Rockbox

synched and updated patch.
now, this patch is some minor fixes to following plugins/files.

*apps/lang/english.lang
in “Set Time/Date” screen, Button A dose not revert setting but POWER dose.

*text_editor
make sure closing file descriptor.
remove code that changes statusbar setting.
*spacerocks
pause only if playing game when HOLD is enabled,
to fix bug that enabling and disabling HOLD in demo allows player to “resume” game.
*star
hide statusbars in menu so as not the bars to be drawn on title.
*rocklife
check result of open properly.
open in Rockbox seems to return negative value not only -1 when error.
iirc, open returns -2 when too many files are opened.
*pictureflow
it seems to returning wrong variable.

*apps/plugins/lib/helper.c
no need to check if rb is NULL since it should be initialized in loader.

*dict, stopwatch, keybox, and pegbox
replace ‘ROCKBOX_DIR “/rocks/(apps|games)/…“’ with ‘PLUGIN_(APPS|GAMES)_DIR “/…“‘.

*apps/keymaps/keymap-gigabeat.c
make source code look better.

   minor_fixes.patch (18.7 KiB)

I committed all the parts of this patch I was familiar with. Someone else will have to take a look at the others. I’ve uploaded a patch with the uncommitted changes.

However, in the future, could you separate different changes into different patches? You can post several patches to the same task if you like, and it makes it much easier to review and commit.

   minor_fixes_02.patch (5.9 KiB)

spacerocks.c, lib/helper.c, dict.c, stopwatch.c, keybox.c & pegbox.c look good to me; can’t comment on the others..

Committed those then.

   minor_fixes_03.patch (1.3 KiB)

Thanks for commiting.
> However, in the future, could you separate different changes into different patches? You can post several patches to the same task if you like, and it makes it much easier to review and commit.
I’ll keep in mind, sorry.

details for the remaining part of patch.
*star
In the function do_menu in apps/menu.c, there is a comment as follows.
/* if hide_bars is true, assume parent has been fixed before passed into
* this function, e.g. with viewport_set_defaults(parent, screen, true) */
But viewports in star don’t seem to be fixed. I started star on my gigabeat x and statusbar was displayed over the title after a few seconds. (statusbar in settings was enabled, of course)
I thought easiest way to fix is to set false to hide_bars.

*first hunk of text_editor
if the file conatins neither CR nor LF (i.e. one line or empty), file descriptor will not be closed.
*rest of text_editor
maybe it’s not good to change setting without asking to do so?

*solitaire_delete_save_file.patch
every time i quit solitaire, it tries to delete save file (even if it dosen’t exist) and cause disk access. it is sometimes annoying.

*solitaire_update_screen.patch
since spash was modified to use viewport, whole display is not cleared before “You Won :)” and cards and cursor remain.

*star_no_statusbar.patch
hide statusbar because it is displaied ovarlapping the title of menu if statusbar in settings is enabled.

*text_editor_close_fd.patch
if the file is empty or contains only one line, file descriptor will not be closed when cheking EOL type.

*dont_change_statusbar_setting
remove code that changes statusbar setting from text_editor and properties.
it’s not good to change setting without asking to do so. should use viewportmanager_set_statusbar instead. and plugin loader hides statusbar with it, no need to reset it.

   solitaire_delete_save_file.pa... (0.8 KiB)
   solitaire_update_screen.patch (0.5 KiB)
   star_no_statusbar.patch (0.4 KiB)
   text_editor_close_fd.patch (0.4 KiB)
   dont_change_statusbar_setting... (2 KiB)

The “text_editor_close_fd.patch” has been committed with r21491.

The “dont_change_statusbar_setting.patch” has been committed with r21493.

“solitaire_delete_save_file.patch” has been committed with r21497.

I don’t see any sense in “star_no_statusbar.patch” Everything seems to work without it just fine.

“solitaire_update_screen.patch” has been committed with r21498.