X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=gfx%2Ftext.c;h=f296be78d3ae11560e0f1fbbf43489ba39ae9ee3;hb=46626ce710c12ab2799c63893b09d06831297213;hp=a9f653c4ad233e10965bfc3d6df20746ae0ce195;hpb=0b674e357f620a2e0810cf74f3e66e8072d9bb86;p=bertos.git

diff --git a/gfx/text.c b/gfx/text.c
index a9f653c4..f296be78 100755
--- a/gfx/text.c
+++ b/gfx/text.c
@@ -1,4 +1,4 @@
-/*!
+/**
  * \file
  * <!--
  * Copyright 2003, 2004, 2005 Develer S.r.l. (http://www.develer.com/)
@@ -15,6 +15,18 @@
 
 /*#*
  *#* $Log$
+ *#* Revision 1.13  2006/07/19 12:56:26  bernie
+ *#* Convert to new Doxygen style.
+ *#*
+ *#* Revision 1.12  2006/05/25 23:35:22  bernie
+ *#* Implement correct and faster clipping for algo text.
+ *#*
+ *#* Revision 1.11  2006/05/15 07:21:06  bernie
+ *#* Doxygen fix.
+ *#*
+ *#* Revision 1.10  2006/04/27 05:39:23  bernie
+ *#* Enhance text rendering to arbitrary x,y coords.
+ *#*
  *#* Revision 1.9  2006/04/11 00:08:24  bernie
  *#* text_offset(): New function, but I'm not quite confident with the design.
  *#*
@@ -107,45 +119,39 @@
 #include <cfg/debug.h>
 
 
-/*! ANSI escape sequences flag: true for ESC state on */
+/**
+ * ANSI escape sequences flag: true for ESC state on.
+ *
+ * \todo Move to Bitmap.flags.
+ */
 static bool ansi_mode = false;
 
-// FIXME: move in bitmap?
-static coord_t text_xoff, text_yoff;
-
-void text_offset(Bitmap *bm, coord_t x, coord_t y)
+/**
+ * Move (imaginary) cursor to coordinates specified.
+ */
+void text_setCoord(struct Bitmap *bm, int x, int y)
 {
-	text_xoff = x;
-	text_yoff = y;
+	bm->penX = x;
+	bm->penY = y;
 }
 
-/*!
+
+/**
  * Move (imaginary) cursor to column and row specified.
  * Next text write will start a that row and col.
  */
-void text_moveto(struct Bitmap *bm, int row, int col)
+void text_moveTo(struct Bitmap *bm, int row, int col)
 {
 	ASSERT(col >= 0);
 	ASSERT(col < bm->width / bm->font->width);
 	ASSERT(row >= 0);
 	ASSERT(row < bm->height / bm->font->height);
 
-	bm->penX = col * bm->font->width + text_xoff;
-	bm->penY = row * bm->font->height + text_yoff;
-}
-
-
-/*!
- * Move (imaginary) cursor to coordinates specified.
- */
-void text_setcoord(struct Bitmap *bm, int x, int y)
-{
-	bm->penX = x;
-	bm->penY = y;
+	text_setCoord(bm, col * bm->font->width, row * bm->font->height);
 }
 
 
-/*!
+/**
  * Render char \a c on Bitmap \a bm.
  */
 static int text_putglyph(char c, struct Bitmap *bm)
@@ -169,7 +175,7 @@ static int text_putglyph(char c, struct Bitmap *bm)
 
 	glyph_height = bm->font->height;
 	// FIXME: for vertical fonts only
-	glyph_height_bytes = ROUND_UP2(glyph_height, 8);
+	glyph_height_bytes = (glyph_height + 7) / 8;
 
 	if (bm->font->offset)
 	{
@@ -197,7 +203,19 @@ static int text_putglyph(char c, struct Bitmap *bm)
 		uint8_t styles = bm->styles;
 		uint8_t prev_dots = 0, italic_prev_dots = 0;
 		uint8_t dots;
-		uint8_t row, col;
+		uint8_t row, col, row_bit;
+
+		/*
+		 * To avoid repeating clipping and other expensive computations,
+		 * we cluster calls to gfx_blitRaster() using a small buffer.
+		 */
+		#define CONFIG_TEXT_RENDER_OPTIMIZE 1
+		#if CONFIG_TEXT_RENDER_OPTIMIZE
+			#define RENDER_BUF_WIDTH 12
+			#define RENDER_BUF_HEIGHT 8
+			uint8_t render_buf[RAST_SIZE(RENDER_BUF_WIDTH, RENDER_BUF_HEIGHT)];
+			uint8_t render_xpos = 0;
+		#endif
 
 		/* This style alone could be handled by the fast path too */
 		if (bm->styles & STYLEF_CONDENSED)
@@ -206,28 +224,19 @@ static int text_putglyph(char c, struct Bitmap *bm)
 		if (bm->styles & STYLEF_EXPANDED)
 			glyph_width *= 2;
 
-		/* Check if glyph fits in the bitmap. */
-		if ((bm->penX < 0) || (bm->penX + glyph_width > bm->width)
-			|| (bm->penY < 0) || (bm->penY + glyph_height > bm->height))
-		{
-			kprintf("bad coords x=%d y=%d\n", bm->penX, bm->penY);
-			return 0;
-		}
-
-		for (row = 0; row < glyph_height_bytes; ++row)
+		for (row = 0, row_bit = 0; row < glyph_height_bytes; ++row, row_bit += 8)
 		{
 			/* For each dot column in the glyph... */
 			for (col = 0; col < glyph_width; ++col)
 			{
 				uint8_t src_col = col;
-				uint8_t i;
 
 				/* Expanded style: advances only once every two columns. */
 				if (styles & STYLEF_EXPANDED)
 					src_col /= 2;
 
 				/* Fetch a column of dots from glyph. */
-				dots = PGM_READ_CHAR(RAST_ADDR(glyph, src_col, row * 8, glyph_width));
+				dots = PGM_READ_CHAR(RAST_ADDR(glyph, src_col, row_bit, glyph_width));
 
 				/* Italic: get lower 4 dots from previous column */
 				if (styles & STYLEF_ITALIC)
@@ -248,16 +257,40 @@ static int text_putglyph(char c, struct Bitmap *bm)
 				/* Underlined: turn on base pixel */
 				if ((styles & STYLEF_UNDERLINE)
 					&& (row == glyph_height_bytes - 1))
-					dots |= (1 << (glyph_height - row * 8 - 1));
+					dots |= (1 << (glyph_height - row_bit - 1));
 
 				/* Inverted: invert pixels */
 				if (styles & STYLEF_INVERT)
 					dots = ~dots;
 
 				/* Output dots */
-				for (i = 0; i < 8 && (row * 8) + i < glyph_height; ++i)
-					BM_DRAWPIXEL(bm, bm->penX + col, bm->penY + row * 8 + i, dots & (1<<i));
+				#if CONFIG_TEXT_RENDER_OPTIMIZE
+					render_buf[render_xpos++] = dots;
+					if (render_xpos == RENDER_BUF_WIDTH)
+					{
+						gfx_blitRaster(bm, bm->penX + col - render_xpos + 1, bm->penY + row_bit,
+							render_buf, render_xpos,
+							MIN((uint8_t)RENDER_BUF_HEIGHT, (uint8_t)(glyph_height - row_bit)),
+							RENDER_BUF_WIDTH);
+						render_xpos = 0;
+					}
+				#else
+					gfx_blitRaster(bm, bm->penX + col, bm->penY + row_bit,
+						&dots, 1, MIN((uint8_t)8, glyph_height - row_bit), 1);
+				#endif
 			}
+
+			#if CONFIG_TEXT_RENDER_OPTIMIZE
+				/* Flush out rest of render buffer */
+				if (render_xpos != 0)
+				{
+					gfx_blitRaster(bm, bm->penX + col - render_xpos, bm->penY + row_bit,
+						render_buf, render_xpos,
+						MIN((uint8_t)RENDER_BUF_HEIGHT, (uint8_t)(glyph_height - row_bit)),
+						RENDER_BUF_WIDTH);
+					render_xpos = 0;
+				}
+			#endif
 		}
 	}
 	else
@@ -273,7 +306,7 @@ static int text_putglyph(char c, struct Bitmap *bm)
 }
 
 
-/*!
+/**
  * Render char \c c, with (currently) limited ANSI escapes
  * emulation support and '\n' for newline.
  */
@@ -315,7 +348,7 @@ int text_putchar(char c, struct Bitmap *bm)
 }
 
 
-/*!
+/**
  * Clear the screen and reset cursor position
  */
 void text_clear(struct Bitmap *bmp)