Hi,
why do you change explicit configuration to something implicit?
user have to decide whether to use keyword or keyword-regex and if both given
it's error and was indicated so.
About the case sensitivity, I would like to give a user possibility to decide.
The policy should be:
- if keyword given make it case insensitive (just backwards
compatibility)
- if keyword-regex given it's case sensitive
So the patch should be:
- remove octstr_convert_range from find_translation
- add REG_ICASE for keyword
- documentation
Thanks,
Alexander Malysh
Am 27.08.2010 um 02:24 schrieb Nikos Balkanas:
> Added some needed initialization. Sorry about that.
>
> BR,
> Nikos
> ----- Original Message ----- From: "Nikos Balkanas" <[email protected]>
> To: <[email protected]>
> Cc: "Rene Kluwen" <[email protected]>; "'Alvaro Cornejo'"
> <[email protected]>
> Sent: Friday, August 27, 2010 1:30 AM
> Subject: Re: Patch: gw/urltrans.c
>
>
>> Case insensitivity is observed in the code. It just lacks in configuration.
>> I changed patch to match documentation:
>>
>> 1) When given both keyword-regex and keyword, it will just prefer
>> keyword-regex without complains (according to UG).
>> 2) It will accept any case in configuration for both keyword and
>> keyword-regex and do case incensitive matching.
>> 3) Fixed a diagnostic that was pooping up evrywhere during matches.
>>
>> Enjoy,
>> Nikos
>> ----- Original Message ----- From: "Rene Kluwen" <[email protected]>
>> To: "'Alvaro Cornejo'" <[email protected]>; "'Nikos Balkanas'"
>> <[email protected]>
>> Cc: <[email protected]>
>> Sent: Thursday, August 26, 2010 5:10 PM
>> Subject: RE: Patch: gw/urltrans.c
>>
>>
>> A lot of hand sets, nowadays automatically convert the first letter to
>> uppercase whilst typing an sms message.
>>
>> So I think case-insensivity is not a bad thing?
>>
>> == Rene
>>
>> -----Original Message-----
>> From: [email protected] [mailto:[email protected]] On Behalf
>> Of Alvaro Cornejo
>> Sent: Thursday, 26 August, 2010 15:44
>> To: Nikos Balkanas
>> Cc: [email protected]
>> Subject: Re: Patch: gw/urltrans.c
>>
>> It should then be noted in the user guide. Otherwise we will continue
>> receiving this questions over and over.
>>
>> |---------------------------------------------------------------------------
>> --------------------------------------|
>> Envνe y Reciba Datos y mensajes de Texto (SMS) hacia y desde cualquier
>> celular y Nextel
>> en el Perϊ, Mιxico y en mas de 180 paises. Use aplicaciones 2 vias via
>> SMS y GPRS online
>> Visitenos en www.perusms.NET www.smsglobal.com.mx y
>> www.pravcom.com
>>
>>
>>
>> 2010/8/26 Nikos Balkanas <[email protected]>:
>>> I don't know if this is really necessary. Just using lower case in the
>>> keyword-regex pattern will work as well. Please disregard.
>>>
>>> BR,
>>> Nikos
>>> ----- Original Message ----- From: "Nikos Balkanas" <[email protected]>
>>> To: <[email protected]>
>>> Sent: Thursday, August 26, 2010 8:02 AM
>>> Subject: Patch: gw/urltrans.c
>>>
>>>
>>>> Hi,
>>>>
>>>> Currently keyword-regex is configured to only do exact case matches. This
>>>> is
>>>> in contrast to keyword matching, which is case incensitive. Additionaly,
>>>> input string input string is converted to lower case for matching. This
>>>> will
>>>> cause all keyword-regex patterns with capital letters to fail.
>>>>
>>>> This patch corrects that by inittializing keyword-regex to case
>>>> incensitive
>>>> matching.
>>>> Reported by Mike Cariotoglou
>>>>
>>>> BR,
>>>> Nikos
>>>>
>>>
>>>
>>>
>>
>>
> <urltrans.diff>
