TTextLogAppender is not thread safe

Issue #235 resolved
Honza Rameš created an issue

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)

  1. Cesar Romero

    Line 256 "fLock,Free"

    Shouldn't it be fLock.Leave? fLock.Free will Destroy the fLock instance.

  2. Ryan Potts

    @cesarliws I concur, it appears to me that one should .leave or .release the lock when finished, not .free it.

  3. Log in to comment