odd "rGlyphData.ExtDataRef().mpData != Format::NONE" (vcl/headless/svptext.cxx)

Stephan Bergmann sbergman at redhat.com
Thu Jun 20 07:15:01 PDT 2013


At least current Clang towards 3.4 fails on current master with

> /home/sbergman/lo-clang/core/vcl/headless/svptext.cxx:175:40: error: comparison between pointer and integer ('void *' and 'sal_Int32' (aka 'int'))
>     if( rGlyphData.ExtDataRef().mpData != Format::NONE )
>         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^  ~~~~~~~~~~~~

The code of SvpGlyphPeer::RemovingGlyph appears to be that way since its 
inception in 
<http://cgit.freedesktop.org/libreoffice/core/commit/?id=b6801926b515be4cfb10ca7e3322939f0f97426b> 
"INTEGRATION: CWS mergesvp (1.1.2); FILE ADDED."  Given that 
basebmp::Format::NONE is

> static const sal_Int32 NONE                         = 0;

that might either be a bad way of checking 
rGlyphData.ExtDataRef().mpData is non-null that just happens to work out 
as intended, or might be a broken attempt at checking that 
rGlyphData.ExtDataRef().meInfo is not Format::NONE.

Anybody an idea what might be the better interpretation?

Stephan


More information about the LibreOffice mailing list