[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