<p>Thank you for your contribution, however, there are couple of remarks I want to do:</p>

<ul>
<li>patch is rather hard to follow because along with new code you reformatted the old code, resulting in bigger patch</li>
<li>reformatting the code to be more standard and remove trailing withe spaces is appreciated, but has to be done as a stand alone patch</li>
<li>log_attempt() seems quite redundant to other existing alternatives: xlog() with a particular facility directed to own file or file_write() from corex module in master branch</li>
</ul>

<p>My suggestions:</p>

<ul>
<li>remove addition of log_attempt() (it can still be discussed if you think so, but in that case make a separate pull request for it)</li>
<li>make a patch only for the new functionality with timerec</li>
<li>make a patch for reformatting the code/removing training white spaces</li>
<li>the last two patches can be in the same pull request</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/382#issuecomment-151777052">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZbubFFDrXhXnO34HNqsLdIgDAV3Uks5pAIregaJpZM4GWoH3.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/382#issuecomment-151777052"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>