Attachment 50565 Details for Bug 40344 – does this make sense, don't mistake this for "I know what I'm doing"

[patch] does this make sense, don't mistake this for "I know what I'm doing"

maybe.this.patch (text/plain), 998 bytes, created by Caolán McNamara on 2011-08-25 06:01:25 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Caolán McNamara

Created: 2011-08-25 06:01:25 UTC

Size: 998 bytes

patch

obsolete

>diff --git a/sw/source/core/layout/layact.cxx b/sw/source/core/layout/layact.cxx
>index aa1b561..a2ee91a 100644
>--- a/sw/source/core/layout/layact.cxx
>+++ b/sw/source/core/layout/layact.cxx
>@@ -1794,7 +1794,6 @@ sal_Bool SwLayAction::FormatCntnt( const SwPageFrm *pPage )
>     // In such a case, redraw the whole page to properly update the
>     // header/footer decorators
>     const SwFrm *pLay = pPage->Lower();
>-    bool bPaintWholePage = false;
> 
>     if ( pLay )
>     {
>@@ -1802,13 +1801,10 @@ sal_Bool SwLayAction::FormatCntnt( const SwPageFrm *pPage )
>             pLay = pLay->GetNext();
> 
>         if ( pLay->IsFooterFrm() )
>-        {
>-            bPaintWholePage = true;
>             pImp->GetShell()->AddPaintRect( pPage->Frm() );
>-        }
>     }
> 
>-    while ( !bPaintWholePage && pCntnt && pPage->IsAnLower( pCntnt ) )
>+    while ( pCntnt && pPage->IsAnLower( pCntnt ) )
>     {
>         //Wenn der Cntnt sich eh nicht veraendert koennen wir ein paar
>         //Abkuerzungen nutzen.

Actions: View | Splinter Review

Attachments on bug 40344: 50530 | 50531 | 50532 | 50533 | 50565

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.