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