[Libreoffice] [PATCH] [PUSHED] remove String with OUString
caolanm at redhat.com
Mon Apr 18 13:14:17 PDT 2011
On Tue, 2011-03-22 at 17:22 +0000, Caolán McNamara wrote:
> On Tue, 2011-03-22 at 16:39 +0000, Caolán McNamara wrote:
> > among others
> Yeah, the catch is that there's a hierarchy of virtual methods in there,
> we can't change *some* of the signatures on impls of a specific virtual
> method we need to change all or none. I'll have to revert this for a bit
> because it breaks basic, e.g. make check fails due to various oddities
> from this.
As an aside, with Lubos's addition of -Woverloaded-virtual the offending
methods that changed *some* but not all of the signatures now show
warnings when this happens. So if you want to have another shot at this
the compiler will now tell you when the above problem happens.
More information about the LibreOffice