- Status Closed
- Percent Complete
- Task Type Patches
- Category User Interface
- 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
Attached to Project: Rockbox
Opened by Teruaki Kawashima - 2009-10-25
Last edited by Teruaki Kawashima - 2009-10-27
Opened by Teruaki Kawashima - 2009-10-25
Last edited by Teruaki Kawashima - 2009-10-27
FS#10722 - some improvements for gui/list.c
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.
Closed by Teruaki Kawashima
2009-10-27 13:35
Reason for closing: Accepted
Additional comments about closing:
2009-10-27 13:35
Reason for closing: Accepted
Additional comments about closing:
committed in r23367
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.
I'm sorry. this is one without changes of spaces.
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.
quick fix of gui_synclist_item_is_onscreen. the function doesn't used as far as i greped.