On Friday 10 of February 2012, Stephan Bergmann wrote:
On 02/10/2012 12:18 PM, Lubos Lunak wrote:
This is sal/, so the library needs to maintain binary compatibility,
does it not? That requires adding the OUString overload rather than
fixing the exiting function. I don't know if the const char* overload
really needs the non-const variant as well, but it's a tiny inline
function, so I added it for the symmetry, just in case.
But these are inline functions, so no (sane) client code should be able
to tell the difference if you sneak in the "const." In which case I
would just do it and be done with it.
You're right, I missed that. Since this is a good change on its own, I've
pushed a fix for this.
--
Lubos Lunak
l.lunak@suse.cz
Context
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.