[sr-dev] acc reply reason

[Daniel-Constantin Mierla]

[resent - devel mailing list address was invalid]

For the moment I will just get it walking via headers list, with no 
shortcut in sip_msg_t as hdr_field_t*. The reply doesn't have many 
headers anyhow.

But, if I am not wrong, I think Reason header can occur many times. Or 
at least I saw it two times, one time like:

Reason: Q.850; ...

and in the same message, another one like:

Reason: SIP;...

Overall, I think we should review which shortcut pointers to headers 
makes sense to keep inside sip_msg_t. Some of them are very rare in sip 
messages, probably can be found easily by walking the list of headers 
when needed.

Cheers,
Daniel

On 30/05/14 17:52, Juha Heinanen wrote:
> i'll move this to devel. acc module is not trivial one.
>
> should parsing of reason header be added to parser and struct hdr_field*
> reason to struct sip_mgs?
>
> -- juha

-- 
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda