Similarly, I'd suggest to clean up ValueExprList typedefs, we have different definitions in different places: parser/SelectListFactory.h:typedef std::list<ValueExprPtr> ValueExprList; query/FuncExpr.h:typedef std::list<ValueExprPtr> ValueExprList; query/Predicate.h: typedef std::list<boost::shared_ptr<ValueExpr> > ValueExprList; query/ValueExpr.h:typedef std::list<ValueExprPtr> ValueExprList; Jacek On 03/24/2014 04:42 PM, Jacek Becla wrote: > Daniel > > Suggestion: let's move SelectStmtList from > qana/QueryPlugin.h to query/SelectStmt. > > Ok? Should I open an issue? > > Jacek > > ######################################################################## > 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