From 485bbbdb309f0c343b5d37eb99a453de8116281a Mon Sep 17 00:00:00 2001 From: Nicolas Pennequin Date: Thu, 6 Dec 2007 17:09:24 +0000 Subject: Make the "current line" value in the gradient style code zero-based, and an attempt at more understandable names and better comments. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@15886 a1c6a512-1295-4272-9138-f99709370657 --- firmware/drivers/lcd-16bit.c | 27 +++++++++++++++------------ firmware/export/lcd.h | 4 ++-- 2 files changed, 17 insertions(+), 14 deletions(-) (limited to 'firmware') diff --git a/firmware/drivers/lcd-16bit.c b/firmware/drivers/lcd-16bit.c index ffa673887e..b990f556d3 100644 --- a/firmware/drivers/lcd-16bit.c +++ b/firmware/drivers/lcd-16bit.c @@ -553,11 +553,14 @@ void lcd_gradient_rect(int x1, int x2, int y, int h) #define H_COLOR(lss, lse, cur_line, max_line) \ (((lse) - (lss)) * (cur_line) / (max_line) + (lss)) -/* Fill a rectangle with a gradient for scrolling line */ +/* Fill a rectangle with a gradient for scrolling line. To draw a gradient that + covers several lines, we need to know how many lines will be covered + (the num_lines arg), and which one is the current line within the selection + (the cur_line arg). */ void lcd_gradient_rect_scroll(int x1, int x2, int y, int h, - unsigned char max_line, unsigned char cur_line) + unsigned char num_lines, unsigned char cur_line) { - if (h == 0 || max_line == 0) return; + if (h == 0 || num_lines == 0) return; unsigned tmp_lss = lss_pattern; unsigned tmp_lse = lse_pattern; @@ -568,14 +571,14 @@ void lcd_gradient_rect_scroll(int x1, int x2, int y, int h, int lse_b = (signed)RGB_UNPACK_BLUE(lse_pattern); int lse_g = (signed)RGB_UNPACK_GREEN(lse_pattern); - int h_r = H_COLOR(lss_r, lse_r, cur_line - 1, max_line); - int h_g = H_COLOR(lss_g, lse_g, cur_line - 1, max_line); - int h_b = H_COLOR(lss_b, lse_b, cur_line - 1, max_line); + int h_r = H_COLOR(lss_r, lse_r, cur_line, num_lines); + int h_g = H_COLOR(lss_g, lse_g, cur_line, num_lines); + int h_b = H_COLOR(lss_b, lse_b, cur_line, num_lines); lcd_set_selector_start(LCD_RGBPACK(h_r, h_g, h_b)); - int l_r = H_COLOR(lss_r, lse_r, cur_line, max_line); - int l_g = H_COLOR(lss_g, lse_g, cur_line, max_line); - int l_b = H_COLOR(lss_b, lse_b, cur_line, max_line); + int l_r = H_COLOR(lss_r, lse_r, cur_line+1, num_lines); + int l_g = H_COLOR(lss_g, lse_g, cur_line+1, num_lines); + int l_b = H_COLOR(lss_b, lse_b, cur_line+1, num_lines); lcd_set_selector_end(LCD_RGBPACK(l_r, l_g, l_b)); lcd_gradient_rect(x1, x2, y, h); @@ -898,8 +901,8 @@ void lcd_puts_style_offset(int x, int y, const unsigned char *str, int style, if (style & STYLE_GRADIENT) { drawmode = DRMODE_FG; - if (CURLN_UNPACK(style) == 1) - lcd_gradient_rect(xpos, LCD_WIDTH, ypos, h*MAXLN_UNPACK(style)); + if (CURLN_UNPACK(style) == 0) + lcd_gradient_rect(xpos, LCD_WIDTH, ypos, h*NUMLN_UNPACK(style)); fg_pattern = lst_pattern; } else if (style & STYLE_COLORBAR) { @@ -1067,7 +1070,7 @@ void lcd_scroll_fn(void) /* Gradient line selector */ drawmode = DRMODE_FG; lcd_gradient_rect_scroll(xpos, LCD_WIDTH, ypos, (signed)pf->height, - MAXLN_UNPACK(s->style), + NUMLN_UNPACK(s->style), CURLN_UNPACK(s->style)); fg_pattern = lst_pattern; break; diff --git a/firmware/export/lcd.h b/firmware/export/lcd.h index 056186d5b6..de03222441 100644 --- a/firmware/export/lcd.h +++ b/firmware/export/lcd.h @@ -36,8 +36,8 @@ #define STYLE_MAXLN_MASK 0x000000FF #define CURLN_PACK(x) (((x)<<8) & STYLE_CURLN_MASK) #define CURLN_UNPACK(x) ((unsigned char)(((x)&STYLE_CURLN_MASK) >> 8)) -#define MAXLN_PACK(x) ((x) & STYLE_MAXLN_MASK) -#define MAXLN_UNPACK(x) ((unsigned char)((x) & STYLE_MAXLN_MASK)) +#define NUMLN_PACK(x) ((x) & STYLE_MAXLN_MASK) +#define NUMLN_UNPACK(x) ((unsigned char)((x) & STYLE_MAXLN_MASK)) #endif #ifdef SIMULATOR -- cgit v1.2.3