On 12/17/12 08:47, Alex Hermann wrote:
> On Sunday 16 December 2012, Richard Fuchs wrote:
>
>> + ptr = ptr->next;
>> + while (ptr) {
>> + if (ptr->type == hftype)
>> + break;
>> + ptr = ptr->next;
>> + }
>> + if (!ptr)
>> + break;
>
>
> There is a dedicated function for walking the sibling headers:
> next_sibling_hdr(). Please consider using it instead of open coding the
> loop. Like in attached patch i should have pushed log time ago ;)Thanks for the hint. I was looking for a function that does this (couldn't have been the first time someone needed to do that, right?) but couldn't find it. I'll update the code. cheers
signature.asc
Description: OpenPGP digital signature
_______________________________________________ sr-dev mailing list [email protected] http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
