I updated my patch (1742883) to allow the use of an arbitrary list of Characters, new patch id 1750487.
This is a bit nicer (I hope). I will update the functions in pdc.c once iv tested this a bit more. -----Original Message----- From: Elena-Ramona Modroiu [mailto:[EMAIL PROTECTED] Sent: 27 June 2007 17:53 To: Morgan Gilroy Cc: devel@openser.org Subject: Re: [OpenSER-Devel] Patch submission Hi, the idea is working, but something is missing. PDT_NODE_SIZE should be increased to 16, otherwise it does not work properly. Also the MI functions should be adapted to the new set of accepted characters. With this patch there will be allowed a set of 16 characters: * + , - . / 0 1 2 3 4 5 6 7 8 9 . Five more characters than you need. This also means an increase in the size of the trees (each node having 16 sons), that is the memory allocated. For this reason, I do not like very much this solution. It's working but it's not optimal. I would go for a generalized solution that allows all printable characters and in the same time avoiding to allocate too much memory. For example, making use of a dictionary to define the set of characters allowed in the prefix. This way, PDT_NODE_SIZE will reflect only the needed characters and have a lower value. I will think about if it's worthy ... maybe a better idea will show up. Thanks for the patch and for pointing out this kind of use case. Regards, Ramona Elena-Ramona Modroiu wrote: > OK, I see ... I will have a look at it. > > Regards, > Ramona > > > Morgan Gilroy wrote: >> Yes I saw this before, but unfortunately not all prefixes begin with a >> *, some begin with **, some are just numbers like 75. >> Basically they can be anything including * >> >> -----Original Message----- >> From: Elena-Ramona Modroiu [mailto:[EMAIL PROTECTED] Sent: 26 >> June 2007 08:40 >> To: Morgan Gilroy >> Cc: Klaus Darilion; devel@openser.org >> Subject: Re: [OpenSER-Devel] Patch submission >> >> Hi, >> >> for the needs you describe, just setting in the configuration file >> the "prefix" parameter of the module ( like this: modparam("pdt", >> "prefix", "*")) should do it. >> >> Regards, >> Ramona >> >> >> Morgan Gilroy wrote: >> >>> Thanks, >>> Issue: 1742883 >>> >>> I hope it's alright. >>> (Im new to this :) ) >>> >>> -----Original Message----- >>> From: Klaus Darilion [mailto:[EMAIL PROTECTED] Sent: 25 >>> June 2007 15:17 >>> To: Morgan Gilroy >>> Cc: devel@openser.org >>> Subject: Re: [OpenSER-Devel] Patch submission >>> >>> >>> >>> Morgan Gilroy wrote: >>> >>>> Hi, I have a simple patch to allow the use of * (asterisk) in the PDT >>>> module (at the moment it only allows 0-9 as the first digit), id like >>>> >>> to >>> >>>> submit it. >>>> >>>> What is the process for this? >>>> Just post it on a note on sourceforge.net tracker under patches? >>>> >>> yes >>> >>> >>>> What options should I use for the diffs ? >>>> >>> unified: -u >>> >>> regards >>> klaus >>> >>> ______________________________________________________________________ >>> This email has been scanned by the MessageLabs Email Security System. >>> For more information please visit http://www.messagelabs.com/email >>> ______________________________________________________________________ >>> >>> _______________________________________________ >>> Devel mailing list >>> Devel@openser.org >>> http://openser.org/cgi-bin/mailman/listinfo/devel >>> >> >> >> ______________________________________________________________________ >> This email has been scanned by the MessageLabs Email Security System. >> For more information please visit http://www.messagelabs.com/email >> ______________________________________________________________________ >> > > > _______________________________________________ > Devel mailing list > Devel@openser.org > http://openser.org/cgi-bin/mailman/listinfo/devel ______________________________________________________________________ This email has been scanned by the MessageLabs Email Security System. For more information please visit http://www.messagelabs.com/email ______________________________________________________________________ _______________________________________________ Devel mailing list Devel@openser.org http://openser.org/cgi-bin/mailman/listinfo/devel
