Re: [PATCH 6/6] Remove unnecessary temp variable

Stephan Bergmann <sbergman -AT- redhat.com>

Tue, 19 Jun 2012 09:34:49 +0200

On 06/19/2012 09:09 AM, Brad Sowden wrote:

I believe it's unnecessary. In the pQuickHlpData->Stop(rSh) call that
immediately follows Move() ClearCntnt() is called which then clears
pQuickHlpData->m_aHelpStrings i.e. same result at the eliminated Move()
call.

Excellent. Sorry for not noting the commit message earlier -- I hadn't

grasped the unusual (at least in this project) format where the commit

message is in the message body and only the actual patch content is an

attachment.

Thanks again, Stephan

Context

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.