|
Rockbox mail archiveSubject: Re: Center on loaded theme FS#10391 (+ set_file bugfix FS#10392)Re: Center on loaded theme FS#10391 (+ set_file bugfix FS#10392)
From: Thomas Martitz <thomas.martitz_at_student.htw-berlin.de>
Date: Tue, 30 Jun 2009 00:58:32 +0200 Jonathan Gordon wrote: > > At this point I'm going to bow out of the discussion with this comment.... > *If* the patch does nothing but preselect the last selected config > then I wouldn't exactly be happy, but I'd let it slide... any extra > complication trying to do dirty checking, or reverting or other > weirdness I'd strongly object to on the grounds that the codebase is > difficult enough as it is without making it worse with obscure > features which are not necessary. > You just guess on the complexity, don't you? Where do you exactly see enbearable code complication? BTW, I don't think we need to check against the non-dirty value. Simply mark it dirty once, and keep it until a new theme is loaded. Received on 2009-06-30 Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy |