[Libreoffice] [PUSHED, partial] Re: [PATCH] new cppcheck cleaning in cppu and cppuhelper + error in compilation

David Tardon dtardon at redhat.com
Wed Jan 12 23:06:08 PST 2011


On Wed, Jan 12, 2011 at 01:12:19PM +0000, Caolán McNamara wrote:
> On Tue, 2011-01-11 at 23:17 +0100, Julien Nabet wrote:
> > It's better with the attachments :-)
> 
> Hum, it all looks good except for the operator=, I mean, it compiles but
> I don't think it *should* compile :-) it is changing the state of
> "this". Something odd here.
> 
> Pushed the rest of this however. Need to have another look at that
> operator=
> 

AFAICS the reason is that

inline Mapping( uno_Mapping * pMapping = 0 ) SAL_THROW( () )

is not explicit. So automatic conversion kicks in and the operator= just
calls another instance of itself ;)

D.


More information about the LibreOffice mailing list