diff options
author | Aidan MacDonald <amachronic@protonmail.com> | 2022-10-04 14:02:44 +0100 |
---|---|---|
committer | Aidan MacDonald <amachronic@protonmail.com> | 2022-11-17 16:12:19 +0000 |
commit | 38687821b20dee54bc52718b49b851de5485351d (patch) | |
tree | e725ed25840a183febf958eb32f220f5ec429131 /apps/gui/skin_engine/skin_render.c | |
parent | 7de16eee26e7fe6012ff832c7c8d8fe654f64193 (diff) | |
download | rockbox-38687821b20dee54bc52718b49b851de5485351d.tar.gz rockbox-38687821b20dee54bc52718b49b851de5485351d.zip |
skin engine: Remove playlistviewer viewport field
We already know the viewport at render time (I think... at least
this seems to be the case for themes I tested) so there's no need
to store a pointer to the viewport.
Change-Id: I75fa2262e96c6f735e6b5da33cd4ca9ac68cd2ee
Diffstat (limited to 'apps/gui/skin_engine/skin_render.c')
-rw-r--r-- | apps/gui/skin_engine/skin_render.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/apps/gui/skin_engine/skin_render.c b/apps/gui/skin_engine/skin_render.c index 8184ba8f66..e8f081e7f5 100644 --- a/apps/gui/skin_engine/skin_render.c +++ b/apps/gui/skin_engine/skin_render.c | |||
@@ -937,7 +937,7 @@ void skin_render_playlistviewer(struct playlistviewer* viewer, | |||
937 | struct align_pos * align = &info.align; | 937 | struct align_pos * align = &info.align; |
938 | bool needs_update; | 938 | bool needs_update; |
939 | int cur_pos, start_item, max; | 939 | int cur_pos, start_item, max; |
940 | int nb_lines = viewport_get_nb_lines(SKINOFFSETTOPTR(skin_buffer, viewer->vp)); | 940 | int nb_lines = viewport_get_nb_lines(&skin_viewport->vp); |
941 | #if CONFIG_TUNER | 941 | #if CONFIG_TUNER |
942 | if (get_current_activity() == ACTIVITY_FM) | 942 | if (get_current_activity() == ACTIVITY_FM) |
943 | { | 943 | { |
@@ -983,7 +983,7 @@ void skin_render_playlistviewer(struct playlistviewer* viewer, | |||
983 | /* only update if the line needs to be, and there is something to write */ | 983 | /* only update if the line needs to be, and there is something to write */ |
984 | if (refresh_type && needs_update) | 984 | if (refresh_type && needs_update) |
985 | { | 985 | { |
986 | struct viewport *vp = SKINOFFSETTOPTR(skin_buffer, viewer->vp); | 986 | struct viewport *vp = &skin_viewport->vp; |
987 | if (!info.force_redraw) | 987 | if (!info.force_redraw) |
988 | display->scroll_stop_viewport_rect(vp, | 988 | display->scroll_stop_viewport_rect(vp, |
989 | 0, info.line_number*display->getcharheight(), | 989 | 0, info.line_number*display->getcharheight(), |