[REVIEWED][REVIEW 3-5] fdo#45543 fix RTF import of ms932-encoded characters

Stephan Bergmann sbergman at redhat.com
Tue Mar 13 03:26:56 PDT 2012


On 03/13/2012 09:41 AM, Miklos Vajna wrote:
> On Mon, Mar 12, 2012 at 01:40:07PM +0000, Caolán McNamara<caolanm at redhat.com>  wrote:
>> Doesn't really matter, but in the ctor member init lists, there isn't
>> really a need to call the default ctors explicitly, e.g.
>> , m_aUnicodeBuffer()
>> , m_aHexBuffer()
>> could just be left out
>
> OK, I'll keep that in mind for new code. I just remember at least once I
> left out something similar, got no local warnings, but the OSX -Werror
> tinbuild failed.

IIRC, this sometimes happens with copy ctors, when for some reason 
default initialization of a base or member would be fine (so could be 
left out), but some compilers would warn about the copy ctor not 
explicitly initializing a base or member.

Stephan


More information about the LibreOffice mailing list