Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#11634 - settings_load_config() calls config_save()

Attached to Project: Rockbox
Opened by Marcin Bukat (MarcinBukat) - Monday, 20 September 2010, 09:37 GMT+2
Last edited by Marcin Bukat (MarcinBukat) - Monday, 20 September 2010, 15:42 GMT+2
Task Type Bugs
Category Settings
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.6
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

I found accidentally that settings_load_config() which is called at startup of the rb calls config_save(). This means that config file is rewiriten on every system startup. This can impact flash wear.

look at apps/settings.c:370
This task depends upon

Closed by  Marcin Bukat (MarcinBukat)
Monday, 20 September 2010, 15:42 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  commited as r28120
Comment by Rafaël Carré (funman) - Monday, 20 September 2010, 09:57 GMT+2
Looks like it was added in r2479 for equalizers, and kept in r4772
Comment by Marcin Bukat (MarcinBukat) - Monday, 20 September 2010, 10:49 GMT+2
http://svn.rockbox.org/viewvc.cgi/trunk/apps/settings.c?view=diff&r1=12141&r2=12142

For me it looks like it was introduced in r12142 and never touched from then.
Comment by Marcin Bukat (MarcinBukat) - Monday, 20 September 2010, 11:12 GMT+2
Right this line was added in r2479 and later there ware code refactorings.
Comment by Marcin Bukat (MarcinBukat) - Monday, 20 September 2010, 14:15 GMT+2
I poked around and for me this call is simply not needed any more. Here is the patch that removes it.
   settings_load_config.patch (0.3 KiB) 
 b/apps/settings.c |    1 -
 1 file changed, 1 deletion(-)

Loading...