diff options
author | Nat Goodspeed <nat@lindenlab.com> | 2017-04-27 08:40:01 -0400 |
---|---|---|
committer | Nat Goodspeed <nat@lindenlab.com> | 2017-04-27 08:40:01 -0400 |
commit | c04073da377a222d2d244c94683816c412dfad36 (patch) | |
tree | 831a9108535ddb8edfeae0bbcbc9cc215e33d293 /indra/mac_crash_logger | |
parent | 0d0f2a8aebd8b992764c227daac75c6964052645 (diff) |
DRTVWR-418: Use conventional LLSingleton init/cleanup for LLWinDebug.
LLWinDebug, though an LLSingleton, had (and required explicit calls to)
special init() and cleanup() methods. Kitty Barnett points out that the
cleanup() method was actually being called after LLSingletonBase::deleteAll(),
requiring resurrection of the deleted LLWinDebug, which sometimes led to
crashes. (Resurrecting deleted LLSingletons is always suspect.)
Change LLWinDebug::init() and cleanup() to the conventional initSingleton()
and cleanupSingleton() methods. This eliminates the need to make special
method calls at all. In particular, cleanupSingleton() will be called by the
existing LLSingletonBase::cleanupAll() call near viewer shutdown.
We retain the early LLWinDebug::instance() call, which implicitly initializes
the LLWinDebug instance, because evidently we want that initialized early. But
we no longer require a separate init() call.
Diffstat (limited to 'indra/mac_crash_logger')
0 files changed, 0 insertions, 0 deletions