Daniel, Where should QueryTemplate really live? ./qana/QueryTemplate.cc ./query/QueryTemplate.h Jacek On 03/24/2014 04:47 PM, Jacek Becla wrote: > 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 > ######################################################################## 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