patch for postgresql driver

Lionel Elie Mamane lionel at mamane.lu
Tue Apr 16 18:10:30 PDT 2013


On Tue, Apr 16, 2013 at 09:24:03PM +0100, Wols Lists wrote:
> On 16/04/13 20:28, Lionel Elie Mamane wrote:
>> On Tue, Apr 16, 2013 at 03:16:25PM +0100, Wols Lists wrote:

>>> I can't get my argument casting right - in the original
>>> code, matchIgnoreAsciiCaseAsciiL wraps its argument in
>>> RTL_CONSTASCII_STRINGPARAM.

>> RTL_CONSTASCII_STRINGPARAM is a hack/optimisation specific to
>> compile-time constants. With your change, the string is not anymore
>> a compile-time constant, so just use

>>  equalsIgnoreAsciiCaseAscii()

> I'm puzzled that it's not a compile-time constant

Well, it is not even a run-time constant: at each iteration of the
loop, it is a different value.

> As soon as LO has built successfully, I'll sort out the patch and ask
> someone to test it :-)

The PostgreSQL driver is (particularly) my area, so keep me in CC.

If you are interested in working on the Postgres driver, have a look
at https://bugs.freedesktop.org/show_bug.cgi?id=43369

There are also a few bugs I didn't have time to look at.

-- 
Lionel


More information about the LibreOffice mailing list