<p>Few more things that need to be done:</p>

<ul>
<li>the commit message must have the format as per guidelines at:

<ul>
<li><a href="http://www.kamailio.org/wiki/devel/github-contributions#commit_message_format">http://www.kamailio.org/wiki/devel/github-contributions#commit_message_format</a></li>
</ul>
</li>
<li>this patch seems to just copy a lot of code from topoh module to sipcapture. It means that if there is a change in topoh, it needs to be resync'ed in sipcapture. This is not the right way to do. Better export an intermodule API in topoh and bind to that one from sipcapture. In this way, the code and parameters from topoh will be used by sipcapture. Sipcapture should get a new module parameter to control if it should bind or not to topoh.</li>
</ul>

<p>You can look at sl module how it exports some functions (modules/sl/sl.h, the sl_api_t structure and the sl_load_api() function) and then at registrar module how it binds to sl in mod_init() and use sl functions.</p>

<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/422#issuecomment-160122006">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZaiOV6UYvsgoWjkCb8E3lAooUgk1ks5pKDoLgaJpZM4GqcEe.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/422#issuecomment-160122006"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>