Jacek,
> I wanted to run this by you... I am replacing structures like this:
>
> std::copy(params.begin(), params.end(),
> std::ostream_iterator<std::string>(LOG_STRM(Info),", "));
>
>
> with:
>
> if (LOG_CHECK_LVL(LOG_DEFAULT_NAME(), LOG_LVL_INFO)) {
> std::strstream ss;
> std::copy(params.begin(), params.end(),
> std::ostream_iterator<std::string>(ss,", "));
> LOGF_INFO(ss.str());
> }
I think that's reasonable, and I think we'd do something similar
if we added an official stream API. We could also add a set of macros
to replace the if condition if you're using it a lot. I'd like to see a
space after "ss,".
--
Kian-Tat Lim, LSST Data Management, [log in to unmask]
########################################################################
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
|