Hi Mahesh, I run cppcheck with latest version 1.75. I got from: https://sourceforge.net/projects/cppcheck/files/cppcheck/1.75/ And build, then run with option --enable=performance.
Regards, Canh. -----Original Message----- From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Thursday, September 01, 2016 11:03 AM To: Canh Van Truong; vu.m.ngu...@dektech.com.au; lennart.l...@ericsson.com Cc: opensaf-devel@lists.sourceforge.net Subject: Re: [PATCH 1 of 1] log: fix cppcheck with performance severity [#1975] Hi Canh Van Truong, Can you please share the suggestion reported by cppcheck. -AVM On 8/31/2016 2:19 PM, Canh Van Truong wrote: > osaf/services/saf/logsv/lgs/lgs_config.cc | 35 ++++++++++++------------------ > osaf/services/saf/logsv/lgs/lgs_file.cc | 4 +- > 2 files changed, 16 insertions(+), 23 deletions(-) > > > Fix performance reported by cppcheck. > > diff --git a/osaf/services/saf/logsv/lgs/lgs_config.cc > b/osaf/services/saf/logsv/lgs/lgs_config.cc > --- a/osaf/services/saf/logsv/lgs/lgs_config.cc > +++ b/osaf/services/saf/logsv/lgs/lgs_config.cc > @@ -137,32 +137,25 @@ typedef struct _lgs_conf_t { > lgs_conf_flg_t logDataGroupname_cnfflag; > lgs_conf_flg_t logStreamFileFormat_cnfflag; > > - _lgs_conf_t() { > - /* > - * For the following flags, LGS_CNF_DEF means that no external > - * configuration exists and the corresponding attributes hard-coded > - * default value is used.Is set to false if configuration is found in > - * IMM object or environment variable. > - * See function lgs_logconf_get() for more info. > - */ > + _lgs_conf_t() > + : logRootDirectory(lgs_conf_def.logRootDirectory), > + logRootDirectory_cnfflag(LGS_CNF_DEF), > + logMaxLogrecsize_cnfflag(LGS_CNF_DEF), > + logStreamSystemHighLimit_cnfflag(LGS_CNF_DEF), > + logStreamSystemLowLimit_cnfflag(LGS_CNF_DEF), > + logStreamAppHighLimit_cnfflag(LGS_CNF_DEF), > + logStreamAppLowLimit_cnfflag(LGS_CNF_DEF), > + logMaxApplicationStreams_cnfflag(LGS_CNF_DEF), > + logFileIoTimeout_cnfflag(LGS_CNF_DEF), > + logFileSysConfig_cnfflag(LGS_CNF_DEF), > + logDataGroupname_cnfflag(LGS_CNF_DEF), > + logStreamFileFormat_cnfflag(LGS_CNF_DEF) > + { > OpenSafLogConfig_object_exist = false; > - logRootDirectory_cnfflag = LGS_CNF_DEF; > - logStreamSystemHighLimit_cnfflag = LGS_CNF_DEF; > - logStreamSystemLowLimit_cnfflag = LGS_CNF_DEF; > - logStreamAppHighLimit_cnfflag = LGS_CNF_DEF; > - logStreamAppLowLimit_cnfflag = LGS_CNF_DEF; > - logDataGroupname_cnfflag = LGS_CNF_DEF; > /* > * The following attributes cannot be configured in the config file > * Will be set to false if the attribute exists in the IMM config object > */ > - logMaxLogrecsize_cnfflag = LGS_CNF_DEF; > - logMaxApplicationStreams_cnfflag = LGS_CNF_DEF; > - logFileIoTimeout_cnfflag = LGS_CNF_DEF; > - logFileSysConfig_cnfflag = LGS_CNF_DEF; > - logStreamFileFormat_cnfflag = LGS_CNF_DEF; > - > - logRootDirectory = lgs_conf_def.logRootDirectory; > (void) strcpy(logDataGroupname, lgs_conf_def.logDataGroupname); > (void) strcpy(logStreamFileFormat, lgs_conf_def.logStreamFileFormat); > logMaxLogrecsize = lgs_conf_def.logMaxLogrecsize; diff --git > a/osaf/services/saf/logsv/lgs/lgs_file.cc > b/osaf/services/saf/logsv/lgs/lgs_file.cc > --- a/osaf/services/saf/logsv/lgs/lgs_file.cc > +++ b/osaf/services/saf/logsv/lgs/lgs_file.cc > @@ -51,11 +51,11 @@ struct file_communicate { > size_t outdata_size; > void *outdata_ptr; /* Out data from handlers */ > > - file_communicate() { > + file_communicate() : request_code(LGSF_NOREQ) { > answer_f = false; > request_f = false; > timeout_f = false; > - request_code = LGSF_NOREQ; > return_code = LGSF_NORETC; > indata_ptr = NULL; > outdata_ptr = NULL; ------------------------------------------------------------------------------ _______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel
