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

[Caolán McNamara]

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.

C.