Pass width instead of x+width to ab_draw_markers()/cue_draw_markers() and don't recalculate the width in these functions.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@25053 a1c6a512-1295-4272-9138-f99709370657
This commit is contained in:
parent
1e9789879f
commit
f50885534e
|
@ -165,8 +165,7 @@ void ab_set_B_marker(unsigned int song_position)
|
|||
static inline int ab_calc_mark_x_pos(int mark, int capacity,
|
||||
int offset, int size)
|
||||
{
|
||||
int w = size - offset;
|
||||
return offset + ( (w * mark) / capacity );
|
||||
return offset + ( (size * mark) / capacity );
|
||||
}
|
||||
|
||||
static inline void ab_draw_vertical_line_mark(struct screen * screen,
|
||||
|
@ -195,14 +194,14 @@ static inline void ab_draw_arrow_mark(struct screen * screen,
|
|||
}
|
||||
|
||||
void ab_draw_markers(struct screen * screen, int capacity,
|
||||
int x0, int x1, int y, int h)
|
||||
int x, int y, int w, int h)
|
||||
{
|
||||
/* if both markers are set, determine if they're far enough apart
|
||||
to draw arrows */
|
||||
if ( ab_A_marker_set() && ab_B_marker_set() )
|
||||
{
|
||||
int xa = ab_calc_mark_x_pos(ab_A_marker, capacity, x0, x1);
|
||||
int xb = ab_calc_mark_x_pos(ab_B_marker, capacity, x0, x1);
|
||||
int xa = ab_calc_mark_x_pos(ab_A_marker, capacity, x, w);
|
||||
int xb = ab_calc_mark_x_pos(ab_B_marker, capacity, x, w);
|
||||
int arrow_width = (h+1) / 2;
|
||||
if ( (xb-xa) < (arrow_width*2) )
|
||||
{
|
||||
|
@ -219,12 +218,12 @@ void ab_draw_markers(struct screen * screen, int capacity,
|
|||
{
|
||||
if (ab_A_marker_set())
|
||||
{
|
||||
int xa = ab_calc_mark_x_pos(ab_A_marker, capacity, x0, x1);
|
||||
int xa = ab_calc_mark_x_pos(ab_A_marker, capacity, x, w);
|
||||
ab_draw_arrow_mark(screen, xa, y, h, DIRECTION_RIGHT);
|
||||
}
|
||||
if (ab_B_marker_set())
|
||||
{
|
||||
int xb = ab_calc_mark_x_pos(ab_B_marker, capacity, x0, x1);
|
||||
int xb = ab_calc_mark_x_pos(ab_B_marker, capacity, x, w);
|
||||
ab_draw_arrow_mark(screen, xb, y, h, DIRECTION_LEFT);
|
||||
}
|
||||
}
|
||||
|
|
|
@ -49,7 +49,7 @@ void ab_end_of_track_report(void);
|
|||
#ifdef HAVE_LCD_BITMAP
|
||||
#include "screen_access.h"
|
||||
void ab_draw_markers(struct screen * screen, int capacity,
|
||||
int x0, int x1, int y, int h);
|
||||
int x, int y, int w, int h);
|
||||
#endif
|
||||
|
||||
/* These functions really need to be inlined for speed */
|
||||
|
|
|
@ -381,16 +381,15 @@ static inline void draw_veritcal_line_mark(struct screen * screen,
|
|||
}
|
||||
|
||||
/* draw the cuesheet markers for a track of length "tracklen",
|
||||
between (x1,y) and (x2,y) */
|
||||
between (x,y) and (x+w,y) */
|
||||
void cue_draw_markers(struct screen *screen, struct cuesheet *cue,
|
||||
unsigned long tracklen,
|
||||
int x1, int x2, int y, int h)
|
||||
int x, int y, int w, int h)
|
||||
{
|
||||
int i,xi;
|
||||
int w = x2 - x1;
|
||||
for (i=1; i < cue->track_count; i++)
|
||||
{
|
||||
xi = x1 + (w * cue->tracks[i].offset)/tracklen;
|
||||
xi = x + (w * cue->tracks[i].offset)/tracklen;
|
||||
draw_veritcal_line_mark(screen, xi, y, h);
|
||||
}
|
||||
}
|
||||
|
|
|
@ -79,7 +79,7 @@ bool curr_cuesheet_skip(struct cuesheet *cue, int direction, unsigned long curr_
|
|||
/* draw track markers on the progressbar */
|
||||
void cue_draw_markers(struct screen *screen, struct cuesheet *cue,
|
||||
unsigned long tracklen,
|
||||
int x1, int x2, int y, int h);
|
||||
int x, int y, int w, int h);
|
||||
#endif
|
||||
|
||||
/* check if the subtrack has changed */
|
||||
|
|
|
@ -124,14 +124,14 @@ static void draw_progressbar(struct gui_wps *gwps,
|
|||
int y = pb->y, height = pb->height;
|
||||
unsigned long length, elapsed;
|
||||
|
||||
if (pb->height < 0 && !pb->have_bitmap_pb)
|
||||
if (height < 0)
|
||||
height = font_get(wps_vp->vp.font)->height;
|
||||
|
||||
if (y < 0)
|
||||
{
|
||||
int line_height = font_get(wps_vp->vp.font)->height;
|
||||
/* center the pb in the line, but only if the line is higher than the pb */
|
||||
int center = (line_height-pb->height)/2;
|
||||
int center = (line_height-height)/2;
|
||||
/* if Y was not set calculate by font height,Y is -line_number-1 */
|
||||
y = (-y -1)*line_height + (0 > center ? 0 : center);
|
||||
}
|
||||
|
@ -160,12 +160,12 @@ static void draw_progressbar(struct gui_wps *gwps,
|
|||
#ifdef AB_REPEAT_ENABLE
|
||||
if (ab_repeat_mode_enabled())
|
||||
ab_draw_markers(display, id3->length,
|
||||
pb->x, pb->x + pb->width, y, height);
|
||||
pb->x, y, pb->width, height);
|
||||
#endif
|
||||
|
||||
if (id3->cuesheet)
|
||||
cue_draw_markers(display, id3->cuesheet, id3->length,
|
||||
pb->x, pb->x + pb->width, y+1, height-2);
|
||||
pb->x, y+1, pb->width, height-2);
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -194,7 +194,7 @@ bool curr_cuesheet_skip(int direction, unsigned long curr_pos){return false;}
|
|||
#ifdef HAVE_LCD_BITMAP
|
||||
/* draw track markers on the progressbar */
|
||||
void cue_draw_markers(struct screen *screen, unsigned long tracklen,
|
||||
int x1, int x2, int y, int h){}
|
||||
int x, int y, int w, int h){}
|
||||
#endif
|
||||
|
||||
#ifdef HAVE_ALBUMART
|
||||
|
@ -383,7 +383,7 @@ long default_event_handler(long event)
|
|||
}
|
||||
|
||||
void ab_draw_markers(struct screen * screen, int capacity,
|
||||
int x0, int x1, int y, int h)
|
||||
int x, int y, int w, int h)
|
||||
{
|
||||
}
|
||||
void pcmbuf_beep(unsigned int frequency, size_t duration, int amplitude){}
|
||||
|
|
Loading…
Reference in New Issue
Block a user