John, Can you open an issue and capture that? Thanks, Jacek On 02/19/2015 03:08 PM, John Gates wrote: > I loaded the code into eclipse and there are 28 warnings which should be > fixed or suppressed. There are lot of uninitialized member variables in > constructors (currently my favorite reason to switch to C++ 11) A > ticket should probably be made to do the work and I expect it would go > pretty fast, and many of these are clearly errors. > > Suppressing a warning for a single line of code is not very pretty, but > is probably worth it for 'missing break intentional' warnings. We could > then treat warnings as errors. > There's information about suppressing a warning from a single line or > block of code here: > https://gcc.gnu.org/onlinedocs/gcc/Diagnostic-Pragmas.html > > > The warnings: > > Catching by reference is recommended 'UserQueryError' UserQuery.cc > /qserv_a/core/modules/ccontrol line 247 Code Analysis Problem > > Class 'Checker' has virtual method 'operator ()' but non-virtual > destructor ResourceUnit.h /qserv_a/core/modules/global line > 103 Code Analysis Problem > > Class 'ResRowBuffer' has virtual method 'fetch' but non-virtual > destructor RowBuffer.cc /qserv_a/core/modules/mysql line 110 > Code Analysis Problem > > Class 'RowBuffer' has virtual method 'fetch' but non-virtual > destructor RowBuffer.h /qserv_a/core/modules/mysql line 68 Code > Analysis Problem > > Invalid project path: Include path not found > (/afs/slac.stanford.edu/u/sf/jgates/work/qserv/$LOG_DIR/include). > qserv_a pathentry Path Entry Problem > > Member '_dirty' was not initialized in this constructor > QuerySession.h /qserv_a/core/modules/qproc line 157 Code > Analysis Problem > > Member '_fieldOffset' was not initialized in this constructor > RowBuffer.cc /qserv_a/core/modules/mysql line 131 Code Analysis > Problem > > Member '_hasChunks' was not initialized in this constructor > QuerySession.h /qserv_a/core/modules/qproc line 157 Code > Analysis Problem > > Member '_hasMerge' was not initialized in this constructor > QuerySession.cc /qserv_a/core/modules/qproc line 86 Code > Analysis Problem > > Member '_hasSubChunks' was not initialized in this constructor > QuerySession.h /qserv_a/core/modules/qproc line 157 Code > Analysis Problem > > Member '_hasSubChunks' was not initialized in this constructor > QuerySession.h /qserv_a/core/modules/qproc line 157 Code > Analysis Problem > > Member '_limit' was not initialized in this constructor SelectStmt.cc > /qserv_a/core/modules/query line 92 Code Analysis Problem > > Member '_order' was not initialized in this constructor > OrderByClause.h /qserv_a/core/modules/query line 66 Code > Analysis Problem > > Member '_request' was not initialized in this constructor > QueryResource.h /qserv_a/core/modules/qdisp line 53 Code > Analysis Problem > > Member '_requestType' was not initialized in this constructor > QservPath.h /qserv_a/core/modules/obsolete line 39 Code > Analysis Problem > > Member '_session' was not initialized in this constructor > QueryResource.h /qserv_a/core/modules/qdisp line 53 Code > Analysis Problem > > Member '_sessionId' was not initialized in this constructor > UserQuery.cc /qserv_a/core/modules/ccontrol line 261 Code > Analysis Problem > > Member 'bufferSize' was not initialized in this constructor > TransactionSpec.h /qserv_a/core/modules/qdisp line 52 Code > Analysis Problem > > Member 'chunkCount' was not initialized in this constructor > QueryContext.h /qserv_a/core/modules/query line 64 Code > Analysis Problem > > Member 'counter' was not initialized in this constructor > testProtocol.cc /qserv_a/core/modules/proto line 47 Code > Analysis Problem > > Member 'metaSession' was not initialized in this constructor > testPlugins.cc /qserv_a/core/modules/qana line 51 Code Analysis > Problem > > Member 'needsMerge' was not initialized in this constructor > QueryContext.h /qserv_a/core/modules/query line 64 Code > Analysis Problem > > No break at the end of case MergingRequester.cc > /qserv_a/core/modules/ccontrol line 113 Code Analysis Problem > > No break at the end of case SchemaFactory.cc > /qserv_a/core/modules/mysql line 168 Code Analysis Problem > > No break at the end of case ValueFactorFactory.cc > /qserv_a/core/modules/parser line 118 Code Analysis Problem > > No break at the end of case ValueFactorFactory.cc > /qserv_a/core/modules/parser line 166 Code Analysis Problem > > Statement has no effect 'GOOGLE_PROTOBUF_VERIFY_VERSION' > testProtocol.cc /qserv_a/core/modules/proto line 112 Code > Analysis Problem > > Suspicious semicolon AggregatePlugin.cc /qserv_a/core/modules/qana > line 83 Code Analysis Problem > > ######################################################################## > Use REPLY-ALL to reply to list > > To unsubscribe from the QSERV-L list, click the following link: > https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1 ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the QSERV-L list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1