Uninitialized variables in TTCCLayout

[supermassive]

Hello,
There is a difference in TTCCLayout definitions between 1.2.x and master branches:
https://github.com/log4cplus/log4cplus/blob/1.2.x/include/log4cplus/layout.h#L187-L190

       bool use_gmtime;
       bool thread_printing;
       bool category_prefixing;
       bool context_printing;

https://github.com/log4cplus/log4cplus/blob/master/include/log4cplus/layout.h#L188-L191

       bool use_gmtime = false;
       bool thread_printing = true;
       bool category_prefixing = true;
       bool context_printing = true;

As the result in 1.2.x when TTCCLayout is initialized by that ctor:

TTCCLayout::TTCCLayout(const log4cplus::helpers::Properties& properties)
    : Layout(properties)
    , dateFormat(properties.getProperty (LOG4CPLUS_TEXT("DateFormat"),
            internal::empty_str))
{
    properties.getBool (use_gmtime, LOG4CPLUS_TEXT("Use_gmtime"));
    properties.getBool (thread_printing, LOG4CPLUS_TEXT("ThreadPrinting"));
    properties.getBool (category_prefixing, LOG4CPLUS_TEXT("CategoryPrefixing"));
    properties.getBool (context_printing, LOG4CPLUS_TEXT("ContextPrinting"));
}

these 4 bools may remain uninitialized in release mode

Upd: VS 2015(update 1), x86, Release.

[wilx]

Thank you for the report. This is now fixed on 1.2.x by merge #149.