OptionRomPkg FrameBufferBltLib: Fix build warning with ICC
authorjljusten <jljusten@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 22 Apr 2011 22:22:05 +0000 (22:22 +0000)
committerjljusten <jljusten@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 22 Apr 2011 22:22:05 +0000 (22:22 +0000)
mBltLibBytesPerPixel is UINTN, and therefore it will always
be >= 0.  Instead an ASSERT is used to verify that MergedMasks
is non-zero.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@11581 6f19259b-4bc3-4df7-8a09-765794883524

OptionRomPkg/Library/FrameBufferBltLib/FrameBufferBltLib.c

index 5bb3cfb..cfd5212 100644 (file)
@@ -65,9 +65,8 @@ ConfigurePixelBitMaskFormat (
   }\r
   MergedMasks = (UINT32) (MergedMasks | Masks[3]);\r
 \r
-  mBltLibBytesPerPixel = HighBitSet32 (MergedMasks);\r
-  ASSERT (mBltLibBytesPerPixel >= 0);\r
-  mBltLibBytesPerPixel = (mBltLibBytesPerPixel + 7) / 8;\r
+  ASSERT (MergedMasks != 0);\r
+  mBltLibBytesPerPixel = (HighBitSet32 (MergedMasks) + 7) / 8;\r
 \r
   DEBUG ((EFI_D_INFO, "Bytes per pixel: %d\n", mBltLibBytesPerPixel));\r
 \r