On 03/24/2014 09:04 PM, Becla, Jacek wrote:
> Daniel
>
> StringMap looks convoluted to me:
Perhaps 'redundant,' but I wouldn't say 'convoluted'. I see that you
feel strongly, so we can put most of util/common.h into
util/stringTypes.h and just include it.
Still, when only StringMap is needed, I'm not convinced it's
problematic--the typedef seems preferable to using
std::map<std::string,std::string>, and having an #include instead really
only moves the visual clutter to another part of the file. I guess
headers lines are easier to grok at a glance.
-Daniel
########################################################################
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
|