Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


On 02/23/2012 12:22 PM, Michael Meeks wrote:
On Thu, 2012-02-23 at 10:56 +0100, Stephan Bergmann wrote:
Yes, fix should be good.  With the newly added rtl::OUString ctor, a
string literal can now implicitly be converted to an rtl::OUString,
which can lead to ambiguities.  In this case, SimpleResMgr has ctors
that take either a char const * or an rtl::OUString const&.  Probably
something that should be cleaned up, leaving SimpleResMgr with just one
of the two ctors.  (And CREATEVERSIONRESMGR_NAME is just a glorious
macro to turn its argument into a string literal, so can safely be dropped.)

        Easy hack created:

        https://bugs.freedesktop.org/show_bug.cgi?id=46501

Ha! I had already fixed that myself (seeing that it has only a handful of uses); push pending.

        We constantly need more of these, as they keep getting fixed
(somehow) ;-) so it'd be great if people can file/create them as/when
they notice code problems during reading.

Yes, you are right. It might have been better to resist the urge to quickly fix it myself, and file an easy hack instead.

Stephan

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.