<p>Hi Henning, in the end we where calling  <code>if_update_stat</code> twice.<br>
This code is only called when retrieving dialog from the DB.</p>
<p>Yes my comment was about removing the optimized version and keeping the simpler one, <code>if_update_stat</code> is is a macro calling several inline functions.<br>
One way of calling it was to using a counter when in order to call <code>if_update_stat</code> only once.<br>
Then as second call to  <code>if_update_stat</code> was added in <code>b0cd09</code> probably because the optimization with the counter was obfuscating things slightly.</p>
<p>I think the optimization are not needed the cost per dialog seems very low.<br>
But we could still select to keep the optimized version if we want, knowing that since 2016 both version where used anyway</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/1566#issuecomment-398094446">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZU6yvsJsAaSoPV2OULZPwZbQRMFPks5t98cfgaJpZM4UqNqM">mute the thread</a>.<img src="https://github.com/notifications/beacon/AF36ZVAgbd9vZzrrxZ0xeDF1DXPhje7mks5t98cfgaJpZM4UqNqM.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/1566#issuecomment-398094446","url":"https://github.com/kamailio/kamailio/pull/1566#issuecomment-398094446","name":"View Pull Request"},"description":"View this Pull Request on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}</script>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@jchavanton in #1566: Hi Henning, in the end we where calling  `if_update_stat` twice.\r\nThis code is only called when retrieving dialog from the DB.\r\n\r\nYes my comment was about removing the optimized version and keeping the simpler one, `if_update_stat` is is a macro calling several inline functions.\r\nOne way of calling it was to using a counter when in order to call `if_update_stat` only once.\r\nThen as second call to  `if_update_stat` was added in `b0cd09` probably because the optimization with the counter was obfuscating things slightly.\r\n\r\nI think the optimization are not needed the cost per dialog seems very low.\r\nBut we could still select to keep the optimized version if we want, knowing that since 2016 both version where used anyway"}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1566#issuecomment-398094446"}}}</script>
<script type="application/ld+json">{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [kamailio/kamailio] [dialog] stats incremented twice (#1566)",
"sections": [
{
"text": "",
"activityTitle": "**Julien Chavanton**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@jchavanton",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"kamailio/kamailio\",\n\"issueId\": 1566,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"kamailio/kamailio\",\n\"pullRequestId\": 1566\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/kamailio/kamailio/pull/1566#issuecomment-398094446"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 346610316\n}"
}
],
"themeColor": "26292E"
}</script>