[Freeswitch-dev] switch_xml_config.c

Mathieu Rene mrene_lists at avgs.ca
Sun Feb 6 09:41:37 MSK 2011


Hi,

It indeed seems appropriate to change those to be inclusive.

Fixed in 0d5fcf65a0fef932f32874da6f4bdddb69279c53

Mathieu Rene
Avant-Garde Solutions Inc
Office: + 1 (514) 664-1044 x100
Cell: +1 (514) 664-1044 x200
mrene at avgs.ca




On 2011-01-26, at 3:24 PM, Gilles Depatie wrote:

> Hi
>  
> I’m running into what appears to be a questionable range interpretation issue in switch_xml_config_parse_event function.
>  
> When enforce_min and enforce_max are TRUE should the value not be inclusive of MIN and MAX?
>  
> Say a value range of min = ‘0’ and max = ‘1000’, should values of 0 and 1000 not be acceptable?
>  
> The current statement:
>                         if ((int_options->enforce_min && !(intval > int_options->min)) || (int_options->enforce_max && !(intval < int_options->max)))
> excludes those values presently
>  
> The error message seems to imply that the range is inclusive of the min and max values
>  
> I’ve had a look at the latest git version or the file and in this respect nothing has changed from the version I am currently using
> GILLES
> Gilles Depatie, Senior S/W Designer, MapleWorks Technology
> O: 819-776-6066 x353  |  gilles.depatie at mapleworks.com
>  
> _______________________________________________
> FreeSWITCH-dev mailing list
> FreeSWITCH-dev at lists.freeswitch.org
> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
> http://www.freeswitch.org

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freeswitch.org/pipermail/freeswitch-dev/attachments/20110206/2243bae2/attachment-0001.html 


More information about the FreeSWITCH-dev mailing list