Cppcheck reports "Logical conjunction always evaluates to false" in text_gfx.cxx
caolanm at redhat.com
Mon Mar 26 07:32:41 PDT 2012
On Mon, 2012-03-26 at 18:03 +0400, Ivan Timofeev wrote:
> Hi Julien,
> On 24.03.2012 03:26, julien2412 wrote:
> > Just for the update, is there anybody who could help about this ?
> I am not expert, too, but looking at vcl/source/gdi/sallayout.cxx:115
> and http://www.unicode.org/charts/PDF/UFF00.pdf I'd say that the
> condition should be at least
> (nChar >= 0xff5b && nChar < 0xff64 )
> However, I do not know why this implementation (in text_gfx.cxx) differs
> from sallayout.cxx.
Quite probably the one in text_gfx.cxx lived in a different toplevel
"psprint" modules ages ago, and got moved into vcl when that module
finally got merged into vcl. There was loads of sort-of-duplicated stuff
between psprint and vcl.
I say we just merge the two things together like so...
More information about the LibreOffice