[PATCH][PUSHED] Fix windows build

Stephan Bergmann sbergman at redhat.com
Thu Feb 23 06:49:18 PST 2012


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


More information about the LibreOffice mailing list