[Libreoffice] [PUSHED] Re: [PATCH] Cpp Cleanliness: unread variable

Caolán McNamara caolanm at redhat.com
Sun Jan 30 13:34:25 PST 2011


On Sun, 2011-01-30 at 01:11 +0100, Kenneth Venken wrote:
> hi,
> 
> this removes unread variable pTargetPage
> from ./impress/sd/source/core/drawdoc2.cxx

It does, but do we know for a fact that it should be

-pTargetPage = GetSdPage(nPage, PK_STANDARD);
and not 
-pTargetPage = GetSdPage(nPage, PK_STANDARD);
+GetSdPage(nPage, PK_STANDARD);

i.e. it might be that GetSdPage has some vital side-effects and that the
call should remain, and only the return value should be discarded.

Anyway reading through it, SdDrawDocument::GetSdPage is defined as a
const method, so it *claims* to have no side-effects, though it calls
into some stuff that creates the page on demand apparently, though
there's already another call in the ::MovePages function being changed
that goes through the same on-demand loader, so yeah, the patch *surely*
is the right one :-)

Thanks for this, now pushed.

C.



More information about the LibreOffice mailing list