[ros-diffs] [fireball] 40329: - Simplify code. Inspired by Evgeniy Boltik's patch. See issue #4326 for more details.

fireball at svn.reactos.org fireball at svn.reactos.org
Thu Apr 2 18:34:12 CEST 2009


Author: fireball
Date: Thu Apr  2 20:34:11 2009
New Revision: 40329

URL: http://svn.reactos.org/svn/reactos?rev=40329&view=rev
Log:
- Simplify code. Inspired by Evgeniy Boltik's patch.
See issue #4326 for more details.

Modified:
    trunk/reactos/subsystems/win32/win32k/dib/dib1bpp.c

Modified: trunk/reactos/subsystems/win32/win32k/dib/dib1bpp.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/dib/dib1bpp.c?rev=40329&r1=40328&r2=40329&view=diff
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/dib/dib1bpp.c [iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/win32k/dib/dib1bpp.c [iso-8859-1] Thu Apr  2 20:34:11 2009
@@ -233,6 +233,7 @@
 BOOLEAN
 DIB_1BPP_BitBltSrcCopy(PBLTINFO BltInfo)
 {
+	ULONG Color;
 	LONG i, j, sx, sy = BltInfo->SourcePoint.y;
 
 	switch ( BltInfo->SourceSurface->iBitmapFormat )
@@ -247,12 +248,8 @@
 			sx = BltInfo->SourcePoint.x;
 			for (i=BltInfo->DestRect.left; i<BltInfo->DestRect.right; i++)
 			{
-				if(XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_4BPP_GetPixel(BltInfo->SourceSurface, sx, sy)) == 0)
-				{
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 0);
-				} else {
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 1);
-				}
+				Color = XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_4BPP_GetPixel(BltInfo->SourceSurface, sx, sy));
+				DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, Color);
 				sx++;
 			}
 			sy++;
@@ -265,12 +262,8 @@
 			sx = BltInfo->SourcePoint.x;
 			for (i=BltInfo->DestRect.left; i<BltInfo->DestRect.right; i++)
 			{
-				if(XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_8BPP_GetPixel(BltInfo->SourceSurface, sx, sy)) == 0)
-				{
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 0);
-				} else {
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 1);
-				}
+				Color = XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_8BPP_GetPixel(BltInfo->SourceSurface, sx, sy));
+				DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, Color);
 				sx++;
 			}
 			sy++;
@@ -283,12 +276,8 @@
 			sx = BltInfo->SourcePoint.x;
 			for (i=BltInfo->DestRect.left; i<BltInfo->DestRect.right; i++)
 			{
-				if(XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_16BPP_GetPixel(BltInfo->SourceSurface, sx, sy)) == 0)
-				{
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 0);
-				} else {
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 1);
-				}
+				Color = XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_16BPP_GetPixel(BltInfo->SourceSurface, sx, sy));
+				DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, Color);
 				sx++;
 			}
 			sy++;
@@ -301,12 +290,8 @@
 			sx = BltInfo->SourcePoint.x;
 			for (i=BltInfo->DestRect.left; i<BltInfo->DestRect.right; i++)
 			{
-				if(XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_24BPP_GetPixel(BltInfo->SourceSurface, sx, sy)) == 0)
-				{
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 0);
-				} else {
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 1);
-				}
+				Color = XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_24BPP_GetPixel(BltInfo->SourceSurface, sx, sy));
+				DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, Color);
 				sx++;
 			}
 			sy++;
@@ -319,12 +304,8 @@
 			sx = BltInfo->SourcePoint.x;
 			for (i=BltInfo->DestRect.left; i<BltInfo->DestRect.right; i++)
 			{
-				if(XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_32BPP_GetPixel(BltInfo->SourceSurface, sx, sy)) == 0)
-				{
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 0);
-				} else {
-					DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, 1);
-				}
+				Color = XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, DIB_32BPP_GetPixel(BltInfo->SourceSurface, sx, sy));
+				DIB_1BPP_PutPixel(BltInfo->DestSurface, i, j, Color);
 				sx++;
 			}
 			sy++;



More information about the Ros-diffs mailing list