> What are you thoughts on using something like TinyExpr from 
> https://github.com/codeplea/tinyexpr ?

I think it's more than a reasonable solution, especially since it's easy to use 
(1 file + 1 header). I only wrote our own quick parser because I didn't want to 
impose too many dependencies, but the fact that `TinyExpr` is zlib licenced 
code is another reason to go with it! All in all, I'm all for the switch to 
`TinyExpr`.

> if ($(var(temp){s.int}) < 0) {
> ...

This looks like a limitation - we definitely need to make `{s.int}` more 
permissive with such input, since it should have easily grabbed the `-2` part, 
even if the input string were `"-2.foobar"`.


---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/788#issuecomment-185199872
_______________________________________________
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Reply via email to