<blockquote>
<p>I am not sure if there is same result now that the lookup of <code>list</code> and <code>xavp</code> is no longer done only once in <code>build_contact()</code>. In previous version the attributes for all contacts were done inside <code>build_contact()</code> using this single lookup. Now the lookup is done for each contact inside the helper function.</p>
<p>I expect is the same, based on quick thinking. If you did the tests and the result is the same in terms of what is stored in xavp and the order of values, then I am fine to merge.</p>
</blockquote>
<p>I'm going to do a unit for that in <a href="http://www.github.com/kamailio/kamailio-tests">http://www.github.com/kamailio/kamailio-tests</a> so We can be sure. Holding this until then.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/2168?email_source=notifications&email_token=ABO7UZKNC3L4VDNJK6S5VBLQX2HELA5CNFSM4JWYCZBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGKHBMY#issuecomment-563376307">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABO7UZMCJWQKZC7I3Q2P6FTQX2HELANCNFSM4JWYCZBA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/ABO7UZOIHJTS5YHK3GTJICTQX2HELA5CNFSM4JWYCZBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGKHBMY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/2168?email_source=notifications\u0026email_token=ABO7UZKNC3L4VDNJK6S5VBLQX2HELA5CNFSM4JWYCZBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGKHBMY#issuecomment-563376307",
"url": "https://github.com/kamailio/kamailio/pull/2168?email_source=notifications\u0026email_token=ABO7UZKNC3L4VDNJK6S5VBLQX2HELA5CNFSM4JWYCZBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGKHBMY#issuecomment-563376307",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>