Re: Patch: gw/urltrans.c

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>