You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openoffice.apache.org by Pavel Janík <Pa...@Janik.cz> on 2012/01/12 17:22:27 UTC

WaE: sw/source/core/doc/notxtfrm.cxx

Hi,

we have two unused variables in sw/source/core/doc/notxtfrm.cxx. Remove or comment or?

diff -ur sw.orig/source/core/doc/notxtfrm.cxx sw/source/core/doc/notxtfrm.cxx
--- sw.orig/source/core/doc/notxtfrm.cxx	2012-01-12 16:35:11.000000000 +0100
+++ sw/source/core/doc/notxtfrm.cxx	2012-01-12 16:35:50.000000000 +0100
@@ -1043,9 +1043,10 @@
                             aAlignedGrfArea.Left(), aAlignedGrfArea.Top(), 
                             aAlignedGrfArea.Right(), aAlignedGrfArea.Bottom());
                         const bool bCropped(aGrfAttr.IsCropped());
+			/*
                         const bool bMirrorHor(aGrfAttr.GetMirrorFlags() & BMP_MIRROR_HORZ);
                         const bool bMirrorVer(aGrfAttr.GetMirrorFlags() & BMP_MIRROR_VERT);
-
+			*/
                         bDone = paintUsingPrimitivesHelper(
                             *pOut,
                             rSvgDataPtr->getPrimitive2DSequence(),

-- 
Pavel Janík




Re: WaE: sw/source/core/doc/notxtfrm.cxx

Posted by Pavel Janík <Pa...@Janik.cz>.
> Thx for the catch.
> 
> Just double-checked with Armin - the variables shall be removed.

Done.
-- 
Pavel Janík




Re: WaE: sw/source/core/doc/notxtfrm.cxx

Posted by Oliver-Rainer Wittmann <or...@googlemail.com>.
Hi Pavel,

On 12.01.2012 17:22, Pavel Janík wrote:
> Hi,
>
> we have two unused variables in sw/source/core/doc/notxtfrm.cxx. Remove or comment or?
>
> diff -ur sw.orig/source/core/doc/notxtfrm.cxx sw/source/core/doc/notxtfrm.cxx
> --- sw.orig/source/core/doc/notxtfrm.cxx	2012-01-12 16:35:11.000000000 +0100
> +++ sw/source/core/doc/notxtfrm.cxx	2012-01-12 16:35:50.000000000 +0100
> @@ -1043,9 +1043,10 @@
>                               aAlignedGrfArea.Left(), aAlignedGrfArea.Top(),
>                               aAlignedGrfArea.Right(), aAlignedGrfArea.Bottom());
>                           const bool bCropped(aGrfAttr.IsCropped());
> +			/*
>                           const bool bMirrorHor(aGrfAttr.GetMirrorFlags()&  BMP_MIRROR_HORZ);
>                           const bool bMirrorVer(aGrfAttr.GetMirrorFlags()&  BMP_MIRROR_VERT);
> -
> +			*/
>                           bDone = paintUsingPrimitivesHelper(
>                               *pOut,
>                               rSvgDataPtr->getPrimitive2DSequence(),
>

Thx for the catch.

Just double-checked with Armin - the variables shall be removed.


Best regards, Oliver.