<p>Is this patch storing something new in accounting record (eg., storing something new in acc database table)?</p>

<p>If not and it is only about SIP header management, then might be better to relocate the code to siputils module. Acc is intended to be the connector to write records to a backend.</p>

<p>Some remarks:</p>

<ul>
<li>change to be a single class of variables: $pcv(name) - having like: $pcv(full), $pcv(value), $pcv(genaddr) -- it is more compact and adding new fields in the future will fit better</li>
<li>use commit message format as suggested at: <a href="http://www.kamailio.org/wiki/devel/git-commit-guidelines#commit_message">http://www.kamailio.org/wiki/devel/git-commit-guidelines#commit_message</a>
</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/kamailio/kamailio/pull/215#issuecomment-113422700">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZeWn_2nhCcEdmdnzzZKCC-C86LK3ks5oU8SHgaJpZM4FGWjr.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
  <div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
    <link itemprop="url" href="https://github.com/kamailio/kamailio/pull/215#issuecomment-113422700"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>