patch for postgresql driver
Lubos Lunak
l.lunak at suse.cz
Tue Apr 16 08:13:17 PDT 2013
On Tuesday 16 of April 2013, Wols Lists wrote:
> The attached patch is not in logerrit (I've yet to set that up), and
> there's an error in it so I'm hoping someone can help. It does a TODO in
> the file, but I can't get my argument casting right - in the original
> code, matchIgnoreAsciiCaseAsciiL wraps its argument in
> RTL_CONSTASCII_STRINGPARAM. This is defined in sal/inc/rtl/string.h and
> passes both the string and its length. So me removing it is obviously a
> mistake, but it's got some integrity checking in there and I just can't
> get it to pass that.
Actually removing it is not a mistake. The macro is a cumbersome
micro-optimization and here it presumably doesn't make any difference. Just
calling matchIgnoreAsciiCase() should do.
Also note that LO has SAL_N_ELEMENTS macro for getting the number of items in
an array.
--
Lubos Lunak
l.lunak at suse.cz
More information about the LibreOffice
mailing list