Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


Hello,

On master sources updated today, cppcheck reported this:
[sw/source/core/text/frmcrsr.cxx:1123]: (style) Variable 'pLine' is assigned
a value that is never used

   1122     const sal_Unicode* pLineString =
GetTxtNode()->GetTxt().getStr();
   1123     pLine += nStt;
   1124 
   1125     UErrorCode nError = U_ZERO_ERROR;
   1126     UBiDi* pBidi = ubidi_openSized( nLen, 0, &nError );
   1127     ubidi_setPara( pBidi, reinterpret_cast<const UChar
*>(pLineString), nLen, nDefaultDir, NULL, &nError ); // UChar != sal_Unicode
in MinGW

See
http://opengrok.libreoffice.org/xref/core/sw/source/core/text/frmcrsr.cxx#1123
Searching in git history pLine gives only this:

commit 70de0c38fd177b3c95837118c5fb35e380c87e90
Author: Jens-Heiner Rechtien <hr@openoffice.org>
Date:   Thu Mar 27 14:45:43 2003 +0000

    MWS_SRX644: migrate branch mws_srx644 -> HEAD

commit 75c0a101618e822dd29947b220b6b75455438ab1
Author: Andreas Martens <ama@openoffice.org>
Date:   Fri Oct 20 13:51:25 2000 +0000

    New: Cursortravelling into dropcaps

Should this line just be removed or should something must be done?

Julien



--
View this message in context: 
http://nabble.documentfoundation.org/Cppcheck-pLine-is-assigned-a-value-that-is-never-used-sw-module-tp4070086.html
Sent from the Dev mailing list archive at Nabble.com.

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.