- Status Closed
- Percent Complete
- Task Type Bugs
- Category Settings
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#9869 - For a setting screen, if the cursor does not move, the selected item's value is invalid.
For the setting screen, I do not move the cursor and I select the current item.
Then, when option_screen() returns, *(int*)setting→setting does not set the current item’s value.
If the cursor moves, and the top item selects, then *(int*)setting→setting sets the current item’s value.
I send this bug’s patch file.
An sample program was prepared.
test_sample.c copies apps/plugins and SOURCES edits.
Then test.sample.rock builds and it runs from Rockbox.
Closed by jdgordon
2009-12-18 06:41
Reason for closing: Not a Bug
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2009-12-18 06:41
Reason for closing: Not a Bug
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Yoshihisa, You need to fix that patch so
you dont need to know if the setting was
canceled, or do your own list
handling.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
the reason its crashing in your patch is because you are setting val to an invalid value… *value in option_screen() does get initialised correctly. If the selection isnt changed, then neither is the value, so whatever the value was before entering the screen it will be when exiting.
To know whether the item had been selected or the selection of the item had been canceled, I set an invalid value as “val” in test_sample.c.
It is not understood whether the selection of the item had been canceled from the return value and arguments of set_option() (or option_screen()).
Therefore, different processing cannot be done whether for the item to have been selected.
Then, I thought whether be able to judge that the selection of the item was canceled if the value of “val” after set_option() was executed was an invalid value.
settings should never have invalid value.. why do you need to know if a setting was canceled?
For
FS#9892, I made the patch that made it jump to the first page when the bookmark was not selected.I found this bug while making the above patch file.
Does something good method exist?