[Libreoffice] [PUSHED] Re: [PATCH] RTL_CONSTASCII_USTRINGPARAM for libs-core/vbahelper

2010-11-22 Thread Caolán McNamara
On Sun, 2010-11-21 at 21:35 +0100, Julien Nabet wrote: > Hello, > > Here's a patch for libs-core/vbahelper > Julien. Reviewed the vbahelper patch, looks good. Pushed this, thanks for you efforts. btw, when you're sending these patches, you're including the content of previous patches in there as

[Libreoffice] [PUSHED] Re: [PATCH] RTL_CONSTASCII_USTRINGPARAM for libs-core/embeddedobj

2010-11-20 Thread Caolán McNamara
On Sat, 2010-11-20 at 13:47 +0100, Julien Nabet wrote: > Hello, > > Here's a patch for libs-core/embeddedobj Thanks for this. Pushed. C. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libre

Re: [Libreoffice] [PUSHED] Re: [PATCH] RTL_CONSTASCII_USTRINGPARAM for libs-core/connectivity/drivers

2010-11-20 Thread Caolán McNamara
On Sun, 2010-11-21 at 02:36 +0900, Takeshi Abe wrote: > Both versions of "i18n" return a QString. Thanks. Yeah, and with an operator to convert to const char*, not a candidate for RTL_CONSTASCII_USTRINGPARAM then. The check I have in the rostrings branch would catch that when integrated once I fig

Re: [Libreoffice] [PUSHED] Re: [PATCH] RTL_CONSTASCII_USTRINGPARAM for libs-core/connectivity/drivers

2010-11-20 Thread Takeshi Abe
Hi Caolán, On Sat, 20 Nov 2010 16:38:45 +, Caolán McNamara wrote: > -(::rtl::OUString::createFromAscii( i18n("Address Book") )); > +(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM( i18n("Address > Book")) )); > > Hmm, what's "i18n" in this context, I can't seem to see a define o

[Libreoffice] [PUSHED] Re: [PATCH] RTL_CONSTASCII_USTRINGPARAM for libs-core/fpicker

2010-11-20 Thread Caolán McNamara
On Sat, 2010-11-20 at 17:50 +0100, Julien Nabet wrote: > Hello, > > Here's a patch for libs-core/fpicker Looks all good. Thanks for this. I did leave out the changes inside the VISTAFILEDIALOG_CHECKED_COMCALL macro. as its inside a define where the parameter is passed into it, so it can't be gu

[Libreoffice] [PUSHED] Re: [PATCH] RTL_CONSTASCII_USTRINGPARAM for libs-core

2010-11-20 Thread Caolán McNamara
On Fri, 2010-11-19 at 23:23 +0100, Julien Nabet wrote: > Hello, > > Here's a patch for libs-core Looks good, top part was duplicate of previous patch. So pushed the extra bits. Thanks for this. When I see an empty sting being created, e.g. OUString::createFromAscii("") best to just replace it wi

[Libreoffice] [PUSHED] Re: [PATCH] RTL_CONSTASCII_USTRINGPARAM for libs-core/connectivity/drivers

2010-11-20 Thread Caolán McNamara
On Fri, 2010-11-19 at 22:53 +0100, Julien Nabet wrote: > Hello, > > Here's patch for the last changes to do that I've seen in > libs-core/connectivity/drivers -(::rtl::OUString::createFromAscii( i18n("Address Book") )); +(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM( i18n("Address