Re: [Powertop] [PATCH] Bug-fix: for html & csv value for CPU Information

Igor Zhbanov <i.zhbanov at samsung.com>
Newsgroups dev.linux.lists.powertop
Message-ID <[email protected]>
Hi Alexandra,

Alexandra Yates wrote:
> Added cast from long-int to string to correct printout for
> "CPU Information" field on html and csv report.
>
> Signed-off-by: Alexandra Yates <alexandra.yates(a)linux.intel.com>
> ---
>   src/report/report.cpp |    4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/report/report.cpp b/src/report/report.cpp
> index 9e94607..6eb2b63 100644
> --- a/src/report/report.cpp
> +++ b/src/report/report.cpp
> @@ -136,7 +136,9 @@ static void system_info(void)
>   	system_data[5].append(str.c_str());
>   	str = cpu_model();
>   	system_data[6]=__("CPU Information");
> -	system_data[7]= sysconf(_SC_NPROCESSORS_ONLN);
> +	stringstream n_proc;;
Double semicolon.
> +	n_proc << sysconf(_SC_NPROCESSORS_ONLN);
> +	system_data[7]= n_proc.str();
>   	system_data[7].append(str.c_str());
>   
>   	str = read_sysfs_string("/etc/system-release");

-- 
Best regards,
Igor Zhbanov,
Expert Software Engineer,
phone: +7 (495) 797 25 00 ext 3981
e-mail: i.zhbanov(a)samsung.com

B2B Solution group, Moscow R&D center, Samsung Electronics
12 Dvintsev street, building 1
127018, Moscow, Russian Federation
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.