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
|