[PATCH] Bug 51309 - Names of DatabaseRanges should not be case sensitive

Mathieu D. call.protected at gmail.com
Tue Oct 30 09:06:38 PDT 2012


HI,

You will find the patch with the correct license statement attached.

Regards,
Mat'

On 10/29/2012 07:22 PM, Markus Mohrhard wrote:
> Hey Mathieu,
>
> 2012/10/26 Mathieu D.<call.protected at gmail.com>:
>> Hi,
>>
>> I submitted a patch for the bug 51309 - Names of DatabaseRanges should not
>> be case sensitive :
>> https://bugs.freedesktop.org/show_bug.cgi?id=51309
>>
>> You will the diff file attached and on the bug's page.
>> https://bugs.freedesktop.org/page.cgi?id=splinter.html&bug=51309&attachment=69115
>>
>> Changes :
>>
>> - ScDBData::less::operator() is modified to compare upper strings
>>
>> - All the calls to NamedDBs::findByName() are now
>> NamedDBs::findByUpperName().
>>
>> - In case of a call to findByUpperName by an argument, the function
>> ScGlobal::pCharClass->uppercase(aName) was used to ensure upper case.
>>
>> - The definitions of FindByName() are deleted.
>>
>>
> The patch looks good. Could you please create the patch with git
> commit -a&&  git format-patch HEAD~1 so that it includes you correctly
> as author.
>
> And please could you send a licence statement like:
>
> All of my past and future contributions to LibreOffice may be licensed
> under the MPL/LGPLv3+
> dual license.
>
> to the Mailing so list so that we can keep track of it at
> http://wiki.documentfoundation.org/Development/Developers
>
> Thanks a lot for your great contribution. Are there any areas that
> interest you? Are you interested in more calc hacking?
>
> Regards,
> Markus


-- 
Mathieu D.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Patch-for-bug-51309-Names-of-DatabaseRanges-should-n.patch
Type: text/x-patch
Size: 15558 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20121030/a27c5cd8/attachment.bin>


More information about the LibreOffice mailing list