(svn r11076) -Fix: MakeTransparent of 32bpp blitter used 0..100; using 0..255 makes it much faster (frosch)
authortruelight
Sun, 09 Sep 2007 21:56:52 +0000
changeset 7552 56aa62dba204
parent 7551 7e76a6be5873
child 7553 63d4424de5d7
(svn r11076) -Fix: MakeTransparent of 32bpp blitter used 0..100; using 0..255 makes it much faster (frosch)
-Fix: ComposeColourXXX could work a tiny bit faster when using 256, not 255 as value to divide with; downside is that it can give alpha errors (frosch)
src/blitter/32bpp_anim.cpp
src/blitter/32bpp_base.hpp
src/blitter/32bpp_simple.cpp
--- a/src/blitter/32bpp_anim.cpp	Sun Sep 09 17:42:59 2007 +0000
+++ b/src/blitter/32bpp_anim.cpp	Sun Sep 09 21:56:52 2007 +0000
@@ -59,7 +59,7 @@
 
 					/* Make the current color a bit more black, so it looks like this image is transparent */
 					if (src->a != 0) {
-						*dst = MakeTransparent(*dst, 75);
+						*dst = MakeTransparent(*dst, 192);
 						*anim = bp->remap[*anim];
 					}
 					break;
@@ -90,7 +90,7 @@
 	if (pal == PALETTE_TO_TRANSPARENT) {
 		do {
 			for (int i = 0; i != width; i++) {
-				*udst = MakeTransparent(*udst, 60);
+				*udst = MakeTransparent(*udst, 154);
 				*anim = 0;
 				udst++;
 				anim++;
--- a/src/blitter/32bpp_base.hpp	Sun Sep 09 17:42:59 2007 +0000
+++ b/src/blitter/32bpp_base.hpp	Sun Sep 09 21:56:52 2007 +0000
@@ -48,15 +48,19 @@
 	 */
 	static inline uint ComposeColourRGBA(uint r, uint g, uint b, uint a, uint current)
 	{
+		if (a == 0) return current;
+		if (a >= 255) return ComposeColour(0xFF, r, g, b);
+
 		uint cr, cg, cb;
 		cr = GB(current, 16, 8);
 		cg = GB(current, 8,  8);
 		cb = GB(current, 0,  8);
 
+		/* The 256 is wrong, it should be 255, but 256 is much faster... */
 		return ComposeColour(0xFF,
-												(r * a + cr * (255 - a)) / 255,
-												(g * a + cg * (255 - a)) / 255,
-												(b * a + cb * (255 - a)) / 255);
+												(r * a + cr * (256 - a)) / 256,
+												(g * a + cg * (256 - a)) / 256,
+												(b * a + cb * (256 - a)) / 256);
 	}
 
 	/**
@@ -64,24 +68,28 @@
 	*/
 	static inline uint ComposeColourPA(uint colour, uint a, uint current)
 	{
+		if (a == 0) return current;
+		if (a >= 255) return (colour | 0xFF000000);
+
 		uint r, g, b, cr, cg, cb;
-		r  = GB(colour,   16, 8);
-		g  = GB(colour,   8,  8);
-		b  = GB(colour,   0,  8);
+		r  = GB(colour,  16, 8);
+		g  = GB(colour,  8,  8);
+		b  = GB(colour,  0,  8);
 		cr = GB(current, 16, 8);
 		cg = GB(current, 8,  8);
 		cb = GB(current, 0,  8);
 
+		/* The 256 is wrong, it should be 255, but 256 is much faster... */
 		return ComposeColour(0xFF,
-												(r * a + cr * (255 - a)) / 255,
-												(g * a + cg * (255 - a)) / 255,
-												(b * a + cb * (255 - a)) / 255);
+												(r * a + cr * (256 - a)) / 256,
+												(g * a + cg * (256 - a)) / 256,
+												(b * a + cb * (256 - a)) / 256);
 	}
 
 	/**
 	* Make a pixel looks like it is transparent.
 	* @param colour the colour already on the screen.
-	* @param amount the amount of transparency, times 100.
+	* @param amount the amount of transparency, times 256.
 	* @return the new colour for the screen.
 	*/
 	static inline uint MakeTransparent(uint colour, uint amount)
@@ -91,7 +99,7 @@
 		g = GB(colour, 8,  8);
 		b = GB(colour, 0,  8);
 
-		return ComposeColour(0xFF, r * amount / 100, g * amount / 100, b * amount / 100);
+		return ComposeColour(0xFF, r * amount / 256, g * amount / 256, b * amount / 256);
 	}
 
 	/**
--- a/src/blitter/32bpp_simple.cpp	Sun Sep 09 17:42:59 2007 +0000
+++ b/src/blitter/32bpp_simple.cpp	Sun Sep 09 21:56:52 2007 +0000
@@ -40,7 +40,7 @@
 					 *  we produce a result the newgrf maker didn't expect ;) */
 
 					/* Make the current color a bit more black, so it looks like this image is transparent */
-					if (src->a != 0) *dst = MakeTransparent(*dst, 75);
+					if (src->a != 0) *dst = MakeTransparent(*dst, 192);
 					break;
 
 				default:
@@ -60,7 +60,7 @@
 	if (pal == PALETTE_TO_TRANSPARENT) {
 		do {
 			for (int i = 0; i != width; i++) {
-				*udst = MakeTransparent(*udst, 60);
+				*udst = MakeTransparent(*udst, 154);
 				udst++;
 			}
 			udst = udst - width + _screen.pitch;