[Libreoffice] [PATCH] [PUSHED] remove String with OUString

Caolán McNamara caolanm at redhat.com
Tue Apr 19 14:09:01 PDT 2011


On Tue, 2011-04-19 at 20:31 +0100, Noel Power wrote:
> <sigh> I am sorry it seems I am reverting your revert ( I didn't see
> your mail at all, only saw it now from searching the archives because I
> didn't see my own mail due some mail client problems :-( ) I committed
> this today ( hopefully with the virtually function related functions
> hunks removed ) at least I didn't get any errors.

My revert was back in March, I only remembered to mention today that the
new compiler warnings would catch this. If the smoketest passes then the
changes are good because the smoketest (being in basic) catches the
error. You *did* run the smoketest didn't you. I'm sure you did.

C.



More information about the LibreOffice mailing list