Rockbox

Tasklist

FS#10722 - some improvements for gui/list.c

Attached to Project: Rockbox
Opened by Teruaki Kawashima (teru) - Sunday, 25 October 2009, 14:56 GMT
Last edited by Teruaki Kawashima (teru) - Tuesday, 27 October 2009, 13:35 GMT
Task Type Patches
Category User Interface
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

this patch is some improvements for gui/list.c
* add function list_get_nb_lines and replace duplicated codes to calculate number of lines by it.
* refine calculation of gui_list->start_item[screen] in function gui_list_put_selection_on_screen.
- tries to show as many items as possible. hopefully fix  FS#10054  without further glitches, iiuc the issue.
* remove trailing spaces and fix indents.
This task depends upon

Closed by  Teruaki Kawashima (teru)
Tuesday, 27 October 2009, 13:35 GMT
Reason for closing:  Accepted
Additional comments about closing:  committed in r23367
Comment by Thomas Martitz (kugel.) - Sunday, 25 October 2009, 18:50 GMT
If you want your patches reviewed (which I assume you do if you upload it here), then please separate indention & space fixes from actual code changes to make reviewing easier.

I'll have a look if nobody beats me to it, but remember the above for future patches.
Comment by Teruaki Kawashima (teru) - Monday, 26 October 2009, 11:05 GMT
I'm sorry. this is one without changes of spaces.
Comment by Thomas Martitz (kugel.) - Monday, 26 October 2009, 11:29 GMT
Thanks! Much easier to review now.

The patch looks very good. I assume it works, so go for it :)

On a unrelated note: gui_synclist_item_is_onscreen() seems to only check the lower bound of the visible list, not the upper. Is that correct? I don't think so. But that's not related to your patch.
Comment by Teruaki Kawashima (teru) - Monday, 26 October 2009, 13:39 GMT
quick fix of gui_synclist_item_is_onscreen. the function doesn't used as far as i greped.

Loading...