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