- changed status to resolved
TTextLogAppender is not thread safe
Issue #235
resolved
TTextLogAppender
implementation is not thread safe even if ErrOutput
is used solely by the logging framework.
Note that if someone else is using ErrOutput
this cannot be thread-safe by design.
Comments (7)
-
reporter -
Line 256 "fLock,Free"
Shouldn't it be fLock.Leave? fLock.Free will Destroy the fLock instance.
-
@cesarliws I concur, it appears to me that one should
.leave
or.release
the lock when finished, not.free
it. -
reporter added thread lock to TTextLogAppender (fixes
#235)→ <<cset 31ca73cab65b>>
-
reporter Thanks guys just a typo. Fixed, please rebase your repos carefully.
-
Thanks for the quick fix.
-
repo owner - changed version to 1.2
- Log in to comment
added thread lock to TTextLogAppender (fixes
#235)→ <<cset 3fa29a2a5e56>>