X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=gfx%2Fbitmap.c;h=d74c7735487c64a2957a8e9c3c83381045162357;hb=3b33848a29f6a9ba49a2913ba6b721e01d8b95f5;hp=01ae977a105d4782e14910d989c3e22abc1c71c9;hpb=39108834d9c102712d6c11830ca9b9fb5b098d19;p=bertos.git

diff --git a/gfx/bitmap.c b/gfx/bitmap.c
index 01ae977a..d74c7735 100755
--- a/gfx/bitmap.c
+++ b/gfx/bitmap.c
@@ -16,6 +16,21 @@
 
 /*#*
  *#* $Log$
+ *#* Revision 1.10  2006/05/27 17:21:15  bernie
+ *#* Factor out, simplify and document clipping.
+ *#*
+ *#* Revision 1.9  2006/05/25 23:35:40  bernie
+ *#* Cleanup.
+ *#*
+ *#* Revision 1.8  2006/03/27 04:48:56  bernie
+ *#* gfx_blitImage(): New function; gfx_blitRaster(): Fix clipping bug.
+ *#*
+ *#* Revision 1.7  2006/03/07 22:18:04  bernie
+ *#* Correctly compute text width for prop fonts; Make styles a per-bitmap attribute.
+ *#*
+ *#* Revision 1.6  2006/02/23 11:17:16  bernie
+ *#* Documentation fixes.
+ *#*
  *#* Revision 1.5  2006/02/15 09:10:15  bernie
  *#* Implement prop fonts; Fix algo styles.
  *#*
@@ -70,6 +85,7 @@ void gfx_bitmapInit(Bitmap *bm, uint8_t *raster, coord_t w, coord_t h)
 
 #if CONFIG_GFX_TEXT
 	gfx_setFont(bm, &default_font);
+	bm->styles = 0;
 #endif
 
 #if CONFIG_GFX_CLIPPING
@@ -85,6 +101,7 @@ void gfx_bitmapInit(Bitmap *bm, uint8_t *raster, coord_t w, coord_t h)
  * Clear the whole bitmap surface to the background color.
  *
  * \note This function does \b not update the current pen position.
+ * \note This function bypasses the current clipping area.
  */
 void gfx_bitmapClear(Bitmap *bm)
 {
@@ -108,6 +125,34 @@ void gfx_blit_P(Bitmap *bm, const pgm_uint8_t *raster)
 }
 #endif /* CPU_HARVARD */
 
+#if CONFIG_GFX_CLIPPING
+	/**
+	 * Clip destination coordinates inside a clipping range.
+	 *
+	 * This macro helps a drawing operation to adjust its
+	 * destination X and Y coordinates inside the destination
+	 * clipping range.
+	 *
+	 * The source start coordinate is adjusted as well
+	 * when destination start clipping occurs.
+	 */
+	#define gfx_clip(dmin, dmax, smin, cmin, cmax) \
+		do { \
+			if ((dmin) < (cmin)) \
+			{ \
+				(smin) += (cmin) - (dmin); \
+				(dmin) = (cmin); \
+			} \
+			(dmax) = MIN((dmax), (cmax)); \
+		} while(0)
+
+#else /* !CONFIG_GFX_CLIPPING */
+
+	#define gfx_clip(dmin, dmax, smin, cmin, cmax) do { } while (0)
+
+#endif /* !CONFIG_GFX_CLIPPING */
+
+
 /**
  * Copy a rectangular area of a bitmap on another bitmap.
  *
@@ -117,8 +162,13 @@ void gfx_blit_P(Bitmap *bm, const pgm_uint8_t *raster)
  *
  * \note Using the same bitmap for \a src and \a dst is unsupported.
  *
- * \param dst Bitmap where the operation writes
- *
+ * \param dst  Bitmap where the operation writes.
+ * \param rect The (xmin;ymin) coordinates provide the top/left offset
+ *             for drawing in the destination bitmap.  If the source
+ *             bitmap is larger than the rectangle, drawing is clipped.
+ * \param src  Bitmap containing the source pixels.
+ * \param srcx Starting X offset in the source bitmap.
+ * \param srcy Starting Y offset in the source bitmap.
  */
 void gfx_blit(Bitmap *dst, const Rect *rect, const Bitmap *src, coord_t srcx, coord_t srcy)
 {
@@ -126,22 +176,19 @@ void gfx_blit(Bitmap *dst, const Rect *rect, const Bitmap *src, coord_t srcx, co
 	coord_t dx, dy, sx, sy;
 
 	/*
-	 * Clip coordinates inside dst->cr and src->width/height.
+	 * Pre-clip coordinates inside src->width/height.
 	 */
 	dxmin = rect->xmin;
-	if (dxmin < dst->cr.xmin)
-	{
-		srcx += dst->cr.xmin - dxmin;
-		dxmin = dst->cr.xmin;
-	}
 	dymin = rect->ymin;
-	if (dymin < dst->cr.ymin)
-	{
-		srcy += dst->cr.ymin - dymin;
-		dymin = dst->cr.ymin;
-	}
-	dxmax = MIN(MIN(rect->xmax, rect->xmin + src->width), dst->cr.xmax);
-	dymax = MIN(MIN(rect->ymax, rect->ymin + src->height), dst->cr.ymax);
+	dxmax = MIN(rect->xmax, rect->xmin + src->width);
+	dymax = MIN(rect->ymax, rect->ymin + src->height);
+
+	/* Perform regular clipping */
+	gfx_clip(dxmin, dxmax, srcx, dst->cr.xmin, dst->cr.xmax);
+	gfx_clip(dymin, dymax, srcy, dst->cr.ymin, dst->cr.ymax);
+
+	//kprintf("dxmin=%d, sxmin=%d, dxmax=%d; ", dxmin, sxmin, dxmax);
+	//kprintf("dymin=%d, symin=%d, dymax=%d\n", dymin, symin, dymax);
 
 	/* TODO: make it not as dog slow as this */
 	for (dx = dxmin, sx = srcx; dx < dxmax; ++dx, ++sx)
@@ -149,28 +196,26 @@ void gfx_blit(Bitmap *dst, const Rect *rect, const Bitmap *src, coord_t srcx, co
 			BM_DRAWPIXEL(dst, dx, dy, BM_READPIXEL(src, sx, sy));
 }
 
-
-void gfx_blitRaster(Bitmap *dst, coord_t dxmin, coord_t dymin, const uint8_t *raster, coord_t w, coord_t h, coord_t stride)
+/**
+ * Blit a raster to a Bitmap.
+ *
+ * \todo Merge this function into gfx_blit()
+ *
+ * \see gfx_blit()
+ */
+void gfx_blitRaster(Bitmap *dst, coord_t dxmin, coord_t dymin,
+		const uint8_t *raster, coord_t w, coord_t h, coord_t stride)
 {
-	coord_t dxmax, dymax;
+	coord_t dxmax = dxmin + w, dymax = dymin + h;
 	coord_t sxmin = 0, symin = 0;
 	coord_t dx, dy, sx, sy;
 
-	/*
-	 * Clip coordinates inside dst->cr.
-	 */
-	if (dxmin < dst->cr.xmin)
-	{
-		sxmin += dst->cr.xmin - dxmin;
-		dxmin = dst->cr.xmin;
-	}
-	if (dymin < dst->cr.ymin)
-	{
-		symin += dst->cr.ymin - dymin;
-		dymin = dst->cr.ymin;
-	}
-	dxmax = MIN(dxmin + w, dst->cr.xmax);
-	dymax = MIN(dymin + h, dst->cr.ymax);
+	/* Perform regular clipping */
+	gfx_clip(dxmin, dxmax, sxmin, dst->cr.xmin, dst->cr.xmax);
+	gfx_clip(dymin, dymax, symin, dst->cr.ymin, dst->cr.ymax);
+
+	//kprintf("dxmin=%d, sxmin=%d, dxmax=%d; ", dxmin, sxmin, dxmax);
+	//kprintf("dymin=%d, symin=%d, dymax=%d\n", dymin, symin, dymax);
 
 	/* TODO: make it not as dog slow as this */
 	for (dx = dxmin, sx = sxmin; dx < dxmax; ++dx, ++sx)
@@ -178,14 +223,29 @@ void gfx_blitRaster(Bitmap *dst, coord_t dxmin, coord_t dymin, const uint8_t *ra
 			BM_DRAWPIXEL(dst, dx, dy, RAST_READPIXEL(raster, sx, sy, stride));
 }
 
+/**
+ * Blit an Image to a Bitmap.
+ *
+ * \see gfx_blit()
+ */
+void gfx_blitImage(Bitmap *dst, coord_t dxmin, coord_t dymin, const Image *image)
+{
+	ASSERT(image);
+
+	gfx_blitRaster(dst, dxmin, dymin,
+			image->raster, image->width, image->height, image->stride);
+}
+
 
-/*!
+#if CONFIG_GFX_CLIPPING
+
+/**
  * Set the bitmap clipping rectangle to the specified coordinates.
  *
  * All drawing performed on the bitmap will be clipped inside this
  * rectangle.
  *
- * \note Following the convention used in all other operations, the
+ * \note Following the convention used for all other operations, the
  *       top-left pixels of the rectangle are included, while the
  *       bottom-right pixels are considered outside the clipping region.
  */
@@ -203,8 +263,9 @@ void gfx_setClipRect(Bitmap *bm, coord_t minx, coord_t miny, coord_t maxx, coord
 	bm->cr.xmax = maxx;
 	bm->cr.ymax = maxy;
 
-/*	kprintf("cr.xmin = %d, cr.ymin = %d, cr.xmax = %d, cr.ymax = %d\n",
-		bm->cr.xMin, bm->cr.ymin, bm->cr.xmax, bm->cr.ymax);
-*/
+//	kprintf("cr.xmin = %d, cr.ymin = %d, cr.xmax = %d, cr.ymax = %d\n",
+//		bm->cr.xMin, bm->cr.ymin, bm->cr.xmax, bm->cr.ymax);
 }
 
+#endif /* CONFIG_GFX_CLIPPING */
+