Hi all, Could you review and signoff the attached patches for inclusion in 3.4 branch?
The first patch is simply some cleanup to make sw build with debug symbols. The second is the fix for https://bugs.freedesktop.org/show_bug.cgi?id=36826 (paragraph borders hidden by paragraph background) Thanks, -- Cédric Bosdonnat LibreOffice hacker http://documentfoundation.org OOo Eclipse Integration developer http://cedric.bosdonnat.free.fr
>From 01ca61d0aacafbe1afa1e72832e5dc1b4989ecd0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Bosdonnat?= <cedric.bosdonnat....@free.fr> Date: Wed, 11 May 2011 14:20:44 +0200 Subject: [PATCH 1/2] fix debug build --- sw/source/core/layout/tabfrm.cxx | 2 +- sw/source/core/undo/SwUndoPageDesc.cxx | 16 ---------------- 2 files changed, 1 insertions(+), 17 deletions(-) diff --git a/sw/source/core/layout/tabfrm.cxx b/sw/source/core/layout/tabfrm.cxx index 93cbfc3..aaa73ca 100644 --- a/sw/source/core/layout/tabfrm.cxx +++ b/sw/source/core/layout/tabfrm.cxx @@ -1568,7 +1568,7 @@ bool SwCntntFrm::CalcLowers( SwLayoutFrm* pLay, const SwLayoutFrm* pDontLeave, } #if OSL_DEBUG_LEVEL > 1 - OSL_FAIL( "LoopControl in SwCntntFrm::CalcLowers" ) + OSL_FAIL( "LoopControl in SwCntntFrm::CalcLowers" ); #endif } } diff --git a/sw/source/core/undo/SwUndoPageDesc.cxx b/sw/source/core/undo/SwUndoPageDesc.cxx index c8f9b0c..7b0eb6a 100644 --- a/sw/source/core/undo/SwUndoPageDesc.cxx +++ b/sw/source/core/undo/SwUndoPageDesc.cxx @@ -64,10 +64,6 @@ void DebugHeaderFooterContent( const SwPageDesc& rPageDesc ) SwFrmFmt* pHeaderFmt = rHead.GetHeaderFmt(); if( pHeaderFmt ) { - nHeaderCount = pHeaderFmt->GetClientCount(); - { - int nHeaderCount = 0; - } const SwFmtCntnt* pCntnt = &pHeaderFmt->GetCntnt(); if( pCntnt->GetCntntIdx() ) nHeaderMaster = pCntnt->GetCntntIdx()->GetIndex(); @@ -77,10 +73,6 @@ void DebugHeaderFooterContent( const SwPageDesc& rPageDesc ) SwFrmFmt* pLeftHeaderFmt = rLeftHead.GetHeaderFmt(); if( pLeftHeaderFmt ) { - nLeftHeaderCount = pLeftHeaderFmt->GetClientCount(); - { - int nLeftHeaderCount = 0; - } const SwFmtCntnt* pLeftCntnt = &pLeftHeaderFmt->GetCntnt(); if( pLeftCntnt->GetCntntIdx() ) nHeaderLeft = pLeftCntnt->GetCntntIdx()->GetIndex(); @@ -93,10 +85,6 @@ void DebugHeaderFooterContent( const SwPageDesc& rPageDesc ) SwFrmFmt* pFooterFmt = rFoot.GetFooterFmt(); if( pFooterFmt ) { - nFooterCount = pFooterFmt->GetClientCount(); - { - int nFooterCount = 0; - } const SwFmtCntnt* pCntnt = &pFooterFmt->GetCntnt(); if( pCntnt->GetCntntIdx() ) nFooterMaster = pCntnt->GetCntntIdx()->GetIndex(); @@ -106,10 +94,6 @@ void DebugHeaderFooterContent( const SwPageDesc& rPageDesc ) SwFrmFmt* pLeftFooterFmt = rLeftFoot.GetFooterFmt(); if( pLeftFooterFmt ) { - nLeftFooterCount = pLeftFooterFmt->GetClientCount(); - { - int nLeftFooterCount = 0; - } const SwFmtCntnt* pLeftCntnt = &pLeftFooterFmt->GetCntnt(); if( pLeftCntnt->GetCntntIdx() ) nFooterLeft = pLeftCntnt->GetCntntIdx()->GetIndex(); -- 1.7.3.4
>From bcc99c69ee6b6ec36f4b9b8563a2594489238e21 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Bosdonnat?= <cedric.bosdonnat....@free.fr> Date: Fri, 13 May 2011 15:25:01 +0200 Subject: [PATCH 2/2] fdo#36826: draw background before borders or borders may be hidden --- sw/source/core/layout/paintfrm.cxx | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/sw/source/core/layout/paintfrm.cxx b/sw/source/core/layout/paintfrm.cxx index 904c2c9..144d2b0 100644 --- a/sw/source/core/layout/paintfrm.cxx +++ b/sw/source/core/layout/paintfrm.cxx @@ -5810,13 +5810,12 @@ void SwFrm::PaintBackground( const SwRect &rRect, const SwPageFrm *pPage, { SwBorderAttrAccess aAccess( SwFrm::GetCache(), (SwFrm*)pFrm ); const SwBorderAttrs &rTmpAttrs = *aAccess.Get(); - /// OD 06.08.2002 #99657# - paint border before painting background - if ( bLowerBorder ) - pFrm->PaintBorder( aBorderRect, pPage, rTmpAttrs ); if ( ( pFrm->IsLayoutFrm() && bLowerBorder ) || aFrmRect.IsOver( aRect ) ) pFrm->PaintBackground( aRect, pPage, rTmpAttrs, bLowMode, bLowerBorder ); + if ( bLowerBorder ) + pFrm->PaintBorder( aBorderRect, pPage, rTmpAttrs ); } pFrm = pFrm->GetNext(); } while ( pFrm && pFrm->GetUpper() == this && -- 1.7.3.4
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice