You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openoffice.apache.org by da...@apache.org on 2015/10/31 14:44:11 UTC

svn commit: r1711631 - /openoffice/trunk/main/basebmp/test/basictest.cxx

Author: damjan
Date: Sat Oct 31 13:44:11 2015
New Revision: 1711631

URL: http://svn.apache.org/viewvc?rev=1711631&view=rev
Log:
Fix a unit test that doesn't compile on MacOS, breaking the build.

Found-by: Pathangi Janardhanan Jatinshravan <JATINSHR001 at e dot ntu dot edu dot sg>
Patch-by: Pathangi Janardhanan Jatinshravan <JATINSHR001 at e dot ntu dot edu dot sg>


Modified:
    openoffice/trunk/main/basebmp/test/basictest.cxx

Modified: openoffice/trunk/main/basebmp/test/basictest.cxx
URL: http://svn.apache.org/viewvc/openoffice/trunk/main/basebmp/test/basictest.cxx?rev=1711631&r1=1711630&r2=1711631&view=diff
==============================================================================
--- openoffice/trunk/main/basebmp/test/basictest.cxx (original)
+++ openoffice/trunk/main/basebmp/test/basictest.cxx Sat Oct 31 13:44:11 2015
@@ -92,7 +92,7 @@ TEST_F(BasicTest, testConstruction)
     ASSERT_TRUE( pDevice->isTopDown() == true ) << "Top down format";
     ASSERT_TRUE( pDevice->getScanlineFormat() == Format::ONE_BIT_MSB_PAL ) << "Scanline format";
     ASSERT_TRUE( pDevice->getScanlineStride() == (aSize2.getY() + 7)/8 ) << "Scanline len";
-    ASSERT_TRUE( pDevice->getPalette() ) << "Palette existence";
+    ASSERT_TRUE( pDevice->getPalette() != NULL ) << "Palette existence";
     ASSERT_TRUE( (*pDevice->getPalette())[0] == Color(0) ) << "Palette entry 0 is black";
     ASSERT_TRUE( (*pDevice->getPalette())[1] == Color(0xFFFFFFFF) ) << "Palette entry 1 is white";
 }