Bug 14417 - Move pref migration code to classes loaded on-demand
: Move pref migration code to classes loaded on-demand
Status: NEW
Product: Logitech Media Server
Classification: Unclassified
Component: Misc
: 7.4.0
: PC Other
: -- normal (vote)
: ---
Assigned To: Unassigned bug - please assign me!
: TinySC
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-09-29 16:14 UTC by Andy Grundman
Modified: 2011-11-06 23:23 UTC (History)
3 users (show)

See Also:
Category: Task


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Grundman 2009-09-29 16:14:06 UTC
There is a lot of pref migration code that is only used once.  This should be factored out into separate classes that can be loaded only if needed.
Comment 1 Adrian Smith 2009-09-30 14:04:44 UTC
Agreed for server prefs.  For client prefs any new client attaching will result in going through all migration steps, so this may be less of a benefit.
Comment 2 Andy Grundman 2009-10-01 07:41:20 UTC
I still think it's worth doing for client prefs, too.  In a typical setup, you won't be adding a new client very often.
Comment 3 Chris Owens 2009-12-18 15:43:49 UTC
Alan is this assigned to you correctly?

All, what real benefit would we see from fixing this bug?  Easier to maintain?
Comment 4 Andy Grundman 2009-12-18 15:55:00 UTC
This is low priority, the only thing we gain is a little bit of memory savings.
Comment 5 Alan Young 2009-12-19 02:02:11 UTC
Yes, it is all about memory. Every chunk of perl code that we can avoid loading in TinySC saves us anothr chunk of memory. I suspect that much of January will be spent looking for memory.
Comment 6 Alan Young 2011-11-06 23:23:27 UTC
Unassigned bugs cannot have a priority.