<html>

    <head>

      <base href="http://bugzilla.slf4j.org/" />

    </head>

    <body>

      <p>

        <div>

            <b><a class="bz_bug_link 

          bz_status_NEW "

   title="NEW --- - Initialization (getILoggerFactory) is not thread safe"

   href="http://bugzilla.slf4j.org/show_bug.cgi?id=176#c29">Comment # 29</a>

              on <a class="bz_bug_link 

          bz_status_NEW "

   title="NEW --- - Initialization (getILoggerFactory) is not thread safe"

   href="http://bugzilla.slf4j.org/show_bug.cgi?id=176">bug 176</a>

              from <span class="vcard"><a class="email" href="mailto:chetan.mehrotra@gmail.com" title="Chetan Mehrotra <chetan.mehrotra@gmail.com>"> <span class="fn">Chetan Mehrotra</span></a>

</span></b>

        <pre><span class="quote">> FWIW the fix in fixSubstitutedLoggers() method isn't really thread-safe either.</span >

As per current code flow it is thread safe. 'fixSubstitutedLoggers' is invoked

*after* the INITIALIZATION_STATE is set to SUCCESSFUL_INITIALIZATION. So by the

time it is invoked TEMP_FACTORY would not be in use and no further

SubstituteLogegrs would be created.

So the only downside is that only during the initialization phase some log

calls would be NOOP and log output would be lost. Similar situtation occurs in

Logback if the Logback config is reset and any log message arrives during that

phase. 

If that also needs to be addressed then we can move just the SimpleLogger to

slf4j-api module and use that in place of NOOP logger for the initial delegate

in SubstituteLogger</pre>

        </div>

      </p>

      <hr>

      <span>You are receiving this mail because:</span>

      <ul>

          <li>You are the assignee for the bug.</li>

      </ul>

    </body>

</html>