I believe the intent is to make sure that the local static Config (in getConfigHelper()) is initialized in a thread safe way (i.e. 2 threads don’t both try to call the constructor). In C++03, that isn’t actually guaranteed, but in C++11, it is (see e.g. http://stackoverflow.com/questions/8102125/is-local-static-variable-initialization-thread-safe-in-c11)

I think what you should do is remove configHelperFla, getConfigHelper() and callOnceHelper(), and change getConfig() to read:

Config & getConfig() {
    static Config c;
    return c;
}

On May 7, 2015, at 10:12 PM, Jacek Becla <[log in to unmask]> wrote:

I'd also welcome help with migrating this line:

boost::once_flag configHelperFlag = BOOST_ONCE_INIT;

(it is in core/modules/wconfig/Config.cc)

std does not have equivalent, so it is a bit
unclear how to migrate it (to me anyway)

Related reading
http://stackoverflow.com/questions/4200813/stdonce-flag-equivalent-of-boost-once-init

Jacek

########################################################################
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


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