using bool in rtl::OUStringBuffer::append

Markus Mohrhard markus.mohrhard at googlemail.com
Wed Oct 3 10:00:54 PDT 2012


Hey,


>>
>> after I got tinderbox mail complaining about ambiguous overload for
>> rtl::OUStringBuffer::append(bool) I wanted to ask if we could add a
>> rtl::OUStringBuffer::append(bool) method or if there are reasons
>> against it. From what I can see it could in the end even share the
>> implementation with the sal_Bool variant just needs an own method in
>> rtl::OUStringBuffer.
>
> the overloads of OUStringBuffer::append are pretty horrible already
> (e.g. append(sal_Bool) vs. append(char) or append(sal_Unicode) which is
> always accidentally invoked when you pass a short);  but adding
> append(bool) won't make that any worse than it is.
>

Seems like append(bool) is not possible without adjusting a lot of
places. The commit seems to have caused some test failures and
crashes. I reverted it for now.

Regards,
Markus


More information about the LibreOffice mailing list