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

[Igor Zhbanov <i.zhbanov at samsung.com>]

[-- Attachment #1: Type: text/plain, Size: 1195 bytes --]

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