Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#12377 - Line Padding in Lists saves incorrect setting in config.cfg

Attached to Project: Rockbox
Opened by Michael Chicoine (mc2739) - Wednesday, 09 November 2011, 07:56 GMT+2
Last edited by Michael Chicoine (mc2739) - Thursday, 10 November 2011, 13:31 GMT+2
Task Type Bugs
Category Settings
Status Closed
Assigned To Thomas Martitz (kugel.)
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Tested on r30933 vx747 sim

When changing the "Line Padding in Lists" setting from Automatic to Off, the setting saved in .rockbox/config.cfg is:

list padding: auto

Changing back to Automatic causes the line to be removed from .rockbox/config.cfg. Manually saving the .cfg file and the setting that is saved is:

list padding: off
This task depends upon

Closed by  Michael Chicoine (mc2739)
Thursday, 10 November 2011, 13:31 GMT+2
Reason for closing:  Fixed
Additional comments about closing:  Fix committed with r30955 (trunk) and r30956 (v3.10 branch)
Comment by Michael Chicoine (mc2739) - Thursday, 10 November 2011, 03:33 GMT+2
This seems to fix the problem.
   settings_list_fix.patch (0.6 KiB) 
 apps/settings_list.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment by Thomas Martitz (kugel.) - Thursday, 10 November 2011, 09:16 GMT+2
That patch is correct.

Loading...