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_parser.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_parser.c')
-rw-r--r-- | apps/gui/skin_engine/skin_parser.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/apps/gui/skin_engine/skin_parser.c b/apps/gui/skin_engine/skin_parser.c index 6fb1b2a43c..e68d5e4d3c 100644 --- a/apps/gui/skin_engine/skin_parser.c +++ b/apps/gui/skin_engine/skin_parser.c | |||
@@ -494,7 +494,6 @@ static int parse_playlistview(struct skin_element *element, | |||
494 | struct playlistviewer *viewer = skin_buffer_alloc(sizeof(*viewer)); | 494 | struct playlistviewer *viewer = skin_buffer_alloc(sizeof(*viewer)); |
495 | if (!viewer) | 495 | if (!viewer) |
496 | return WPS_ERROR_INVALID_PARAM; | 496 | return WPS_ERROR_INVALID_PARAM; |
497 | viewer->vp = PTRTOSKINOFFSET(skin_buffer, &curr_vp->vp); | ||
498 | viewer->show_icons = true; | 497 | viewer->show_icons = true; |
499 | viewer->start_offset = get_param(element, 0)->data.number; | 498 | viewer->start_offset = get_param(element, 0)->data.number; |
500 | viewer->line = PTRTOSKINOFFSET(skin_buffer, get_param_code(element, 1)); | 499 | viewer->line = PTRTOSKINOFFSET(skin_buffer, get_param_code(element, 1)); |