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

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

[Igor Zhbanov]

[-- 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

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

[Alexandra Yates]

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

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;;
+	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");
-- 
1.7.9.5

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

[Alexandra Yates]

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

This patch fixes an error encounterd when printing "CPU Information" 
on html and csv. 

Alexandra Yates (1):
  Bug-fix: for html & csv value for CPU Information

 src/report/report.cpp |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

-- 
1.7.9.5