sal/qa/osl/condition/osl_Condition.cxx | 22 +++++++++++----------- sw/source/core/inc/pagefrm.hxx | 2 +- sw/source/core/layout/paintfrm.cxx | 11 +++++------ 3 files changed, 17 insertions(+), 18 deletions(-)
New commits: commit 283007f8da755af1a9a4a66cc7089aaed3ec8d18 Author: Herbert Dürr <h...@apache.org> Date: Wed Jun 4 11:21:52 2014 +0000 #i125003# fix google-test for osl_condition in the sal module the test didn't initialize the nanosecond part of the condition timeout and so the absolute timeout calculated from the second and nanosecond values could become invalid depending on the pseudo-random value of the nanosecond member. This resulted in spurious gtest failures (first seen on Ubuntu). diff --git a/sal/qa/osl/condition/osl_Condition.cxx b/sal/qa/osl/condition/osl_Condition.cxx index 2feee84..7231d05 100644 --- a/sal/qa/osl/condition/osl_Condition.cxx +++ b/sal/qa/osl/condition/osl_Condition.cxx @@ -216,8 +216,7 @@ namespace osl_Condition Result wait(const TimeValue *pTimeout = 0) */ TEST(Sal_Test_Condition, wait_001) { - TimeValue tv1; - tv1.Seconds = 1; + TimeValue tv1 = {1,0}; ::osl::Condition cond1; ::osl::Condition cond2; @@ -232,14 +231,13 @@ namespace osl_Condition fprintf(stderr,"%d %d %d\n",r1,r2,r3); // #test comment#: test three types of wait. - ASSERT_TRUE( (cond1.wait(&tv1) == ::osl::Condition::result_ok) && - (cond2.wait() == ::osl::Condition::result_ok) && - (cond3.wait(&tv1) == ::osl::Condition::result_timeout) ); + ASSERT_TRUE( cond1.wait(&tv1) == ::osl::Condition::result_ok ); + ASSERT_TRUE( cond2.wait() == ::osl::Condition::result_ok ); + ASSERT_TRUE( cond3.wait(&tv1) == ::osl::Condition::result_timeout ); } TEST(Sal_Test_Condition, wait_002) { - TimeValue tv1; - tv1.Seconds = 1; + TimeValue tv1 = {1,0}; ::osl::Condition aCond; ::osl::Condition::Result wRes, wRes1; @@ -253,9 +251,10 @@ namespace osl_Condition sal_Bool bRes1 = aCond.check( ); // #test comment#: wait a condition after set/reset. - ASSERT_TRUE( !bRes && bRes1 && - ( ::osl::Condition::result_timeout == wRes ) && - ( ::osl::Condition::result_ok == wRes1 ) ); + ASSERT_TRUE( !bRes ); + ASSERT_TRUE( bRes1 ); + ASSERT_TRUE( ::osl::Condition::result_timeout == wRes ); + ASSERT_TRUE( ::osl::Condition::result_ok == wRes1 ); } @@ -288,7 +287,8 @@ namespace osl_Condition sal_Bool bRes1 = aCond.check( ); // #test comment#: use threads to set/reset Condition and check it in main routine. - ASSERT_TRUE( bRes && !bRes1 ); + ASSERT_TRUE( bRes ); + ASSERT_TRUE( !bRes1 ); } } // namespace osl_Condition commit a8a2653568d31a2fb76aeb3183e128e3304452bd Author: Armin Le Grand <a...@apache.org> Date: Wed Jun 4 11:12:30 2014 +0000 Corrected GetDrawBackgrdColor to return a value, not a reference, removed unused var diff --git a/sw/source/core/inc/pagefrm.hxx b/sw/source/core/inc/pagefrm.hxx index 2876704..3b00e71 100644 --- a/sw/source/core/inc/pagefrm.hxx +++ b/sw/source/core/inc/pagefrm.hxx @@ -317,7 +317,7 @@ public: @return reference to an instance of class Color */ - const Color& GetDrawBackgrdColor() const; + const Color GetDrawBackgrdColor() const; /** paint margin area of a page diff --git a/sw/source/core/layout/paintfrm.cxx b/sw/source/core/layout/paintfrm.cxx index 081ad1a..ca5e6ae 100644 --- a/sw/source/core/layout/paintfrm.cxx +++ b/sw/source/core/layout/paintfrm.cxx @@ -308,8 +308,7 @@ void SwCalcPixStatics( OutputDevice *pOut ) //Zum Sichern der statics, damit das Paint (quasi) reentrant wird. class SwSavePaintStatics { - sal_Bool bSFlyMetafile, - bSPageOnly; + sal_Bool bSFlyMetafile; // not used: bSPageOnly; ViewShell *pSGlobalShell; OutputDevice *pSFlyMetafileOut; SwFlyFrm *pSRetoucheFly, @@ -3011,7 +3010,7 @@ SwRootFrm::Paint(SwRect const& rRect, SwPrintData const*const pPrintData) const /// OD 29.08.2002 #102450# /// determine background color of page for <PaintLayer> method /// calls, paint <hell> or <heaven> - const Color aPageBackgrdColor = pPage->GetDrawBackgrdColor(); + const Color aPageBackgrdColor(pPage->GetDrawBackgrdColor()); pPage->PaintBaBo( aPaintRect, pPage, sal_True ); @@ -6489,7 +6488,7 @@ void SwLayoutFrm::RefreshExtraData( const SwRect &rRect ) const @return Color */ -const Color& SwPageFrm::GetDrawBackgrdColor() const +const Color SwPageFrm::GetDrawBackgrdColor() const { const SvxBrushItem* pBrushItem; const Color* pDummyColor; @@ -6605,7 +6604,7 @@ void SwFrm::Retouche( const SwPageFrm * pPage, const SwRect &rRect ) const SwRect aRetouchePart( rRetouche ); if ( aRetouchePart.HasArea() ) { - const Color aPageBackgrdColor = pPage->GetDrawBackgrdColor(); + const Color aPageBackgrdColor(pPage->GetDrawBackgrdColor()); const IDocumentDrawModelAccess* pIDDMA = pSh->getIDocumentDrawModelAccess(); // --> OD #i76669# SwViewObjectContactRedirector aSwRedirector( *pSh ); @@ -6881,7 +6880,7 @@ Graphic SwFlyFrmFmt::MakeGraphic( ImageMap* pMap ) // OD 09.12.2002 #103045# - determine page, fly frame is on const SwPageFrm* pFlyPage = pFly->FindPageFrm(); - const Color aPageBackgrdColor = pFlyPage->GetDrawBackgrdColor(); + const Color aPageBackgrdColor(pFlyPage->GetDrawBackgrdColor()); const IDocumentDrawModelAccess* pIDDMA = pSh->getIDocumentDrawModelAccess(); // --> OD #i76669# SwViewObjectContactRedirector aSwRedirector( *pSh );
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits