[mythtv-commits] mythtv/master commit: 42dfafead by Gavin Hurlbut (Beirdo)

MythTV noreply at mythtv.org
Thu Aug 18 09:41:28 UTC 2011


      Author:  Gavin Hurlbut <ghurlbut at mythtv.org>
 Change Date:  2011-08-18T02:33:53-07:00
   Push Date:  2011/08/18 02:41:20 -0700
  Repository:  mythtv
      Branch:  master
New Revision:  42dfafead8c92dc20e37d4593e75358b6e11f12c
   Changeset:  https://github.com/MythTV/mythtv/commit/42dfafead

Log:

Fix database logging shutdown

It seems that some of the recent changes have caused issues with the shutdown
of the database logging thread.  The main part of this is that the database
connection was still in scope and deemed active when it was being torn down.
This has been changed.  Also, the original intent (to drain the db queue into
to database on shutdown) has been restored.  Also, the wait at startup was
too long for such short-lived programs as mythpreviewgen, and has been
shortened to not wait after the database is ready.  This may need subsequent
tweaking.

Finally, the call to logStop() in mythcorecontext.cpp has been disabled as it
runs after the call to logStop() in the mythcontext dtor.  There may be some
situations that required this second call, so we will have to keep our eyes
peeled.

Modified:

   mythtv/libs/libmythbase/logging.cpp
   mythtv/libs/libmythbase/logging.h
   mythtv/libs/libmythbase/mythcorecontext.cpp



More information about the mythtv-commits mailing list