14 Mar
2013
14 Mar
'13
6:46 p.m.
Hello,
you can move to the core as long as it is licensed as BSD contribution
-- it should be no problem/conflict with parse_supported from kcore, it
was on my todo list, as I spoke with Andreas some time ago and he agreed
to this change.
My only suggestion would be to keep the common pattern on file name for
parser/*, thus naming the new file parse_XYZ.{c,h}
Cheers,
Daniel
On 3/14/13 12:57 PM, Peter Dunkley wrote:
Hello,
I have been working on some outbound related fixes (problems
identified by Olle during SIPit). Some of these require me to parse
the "Require:" header.
There is already a parser for "Supported:" in "lib/kcore" and the
parsing for "Require:" is almost identical. To this end I have
abstracted a lot of the code from "parse_supported.[ch]" into new
"options-tags.[ch]" files and have a much shorter
"parse_supported.[ch]" and "parse_require.[ch]" that use the
options-tags parser.
Another advantage of this is that it will simplify the writing of
proper parsers for other similar headers in HTTP (for example, the
"Sec-WebSocket-Protocol:" header used during the WebSocket handshake.
Now that I've done this I would like to move these parser files from
"lib/kcore" into "parser". Is there any reason I shouldn't do
this?
The code can be found in the pd/outbound branch.
Regards,
Peter
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
--
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Kamailio World Conference, April 16-17, 2013, Berlin
- http://conference.kamailio.com -