id,summary,reporter,owner,description,type,status,priority,milestone,component,version,severity,resolution,keywords,cc,mlocked 7616,Revert settings cache behaviour introduced in [20828],danielk,danielk,"We stopped caching default values, which caused a noticeable performance regression in playback startup. I'm attaching two patches to this ticket, one for trunk, the other for 0.22-fixes. I asked Michael T. Dean about this and the main problem this was trying to address was the caching of defaults when Get{,Num,Float}Setting was called without a default. Then """", 0, and 0.0 would get cached. To address this problem without the performance implications of not caching defaults at all we use a sentinel value of """" and never cache that value. The secondary problem this tried to solve was Get{,Num,Float}Settings called with different defaults in two or more places, this coding error should not cause the types of problems that caching """", 0, and 0.0 can cause since anyone providing a default should be picking a sane one. Note: There is a bit of unneeded redundancy in the settings code, I will be addressing that in trunk in separate patch.",patch,closed,minor,0.23,MythTV - General,head,medium,fixed,,,0