[mythtv-commits] Ticket #6161: Settings code cleanup
MythTV
mythtv at cvs.mythtv.org
Fri Jan 23 11:19:33 UTC 2009
#6161: Settings code cleanup
-----------------------------------------------+----------------------------
Reporter: Matthew Wire <devel at mrwire.co.uk> | Owner: ijr
Type: patch | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - General | Version: head
Severity: low | Mlocked: 0
-----------------------------------------------+----------------------------
Remove code duplication within mythdb. Merge almost identical GetSetting
and GetSettingOnHost code and add wrapper method for GetGlobalSetting
using the same code.[[BR]]
Add doxygen comments for most methods.[[BR]]
Add wrapper for SaveGlobalSetting.[[BR]]
Mythshutdown patch converts mythshutdown to use functions from mythdb
rather than it's own copies.
Danielk was concerned about destabilising a core method, which I do
understand. However, I have been running this code for six months now
without issue and reducing the number of copies of the GetSettings code
from 3 to 1 can only aid stability in the future.
''Versions of these patches were originally posted to #3910''
''See http://www.gossamer-threads.com/lists/mythtv/dev/366009 for more
information''
--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6161>
MythTV <http://www.mythtv.org/>
MythTV
More information about the mythtv-commits
mailing list