On Mar 25, 2014, at 11:45 AM, Daniel L. Wang <[log in to unmask]> wrote: > On 03/24/2014 08:03 PM, Kian-Tat Lim wrote: >> >> Under DRY ("Don't Repeat Yourself"), you can have a >> ValueExprInterface.h that contains just: >> >> class ValueExpr; >> typedef boost::shared_ptr<ValueExpr> ValueExprPtr; >> typedef std::list<ValueExprPtr> ValueExprList; >> >> and include that in ValueExpr.h. >> >> I would not group all such interfaces, particularly if they're >> unrelated, into a single header. > Oooh, this is a great idea. Let's go with this one. +1. My only other comment is that I wish this typedef were named ValueExprPtrList. ######################################################################## 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