<p>This has implications on other parts, such as variables where they can be accessed and set via $mf (iirc).</p>
<p>On the other hand, I don't think having a variable with different lengths is a good approach, because it can break easily config behaviour when moving from one server to another. It should be same size on 32b or 64b, so if a change to increase the size is to be done, then should be <code>long long</code> to ensure it is 64 on all archs. An alternative would be to make the field inside sip_msg_t an array of flag_t values, so can be one or more, even beyond 64b. But this change will have also impact in other parts, therefore would need careful review.</p>
<p>As an alternative for now would be using an avp or xavps, and do bitwise operations (<code>|</code> and <code>&</code>) to set or test.</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/issues/1288#issuecomment-340394835">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZRCkYL0uNLAalGBRy-T9Bo9eDVOTks5sxZwmgaJpZM4QKR-M">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZTyVBZo-LAB1G9nXurElJ1ZLQhBRks5sxZwmgaJpZM4QKR-M.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/issues/1288#issuecomment-340394835"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>

<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://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@miconda in #1288: This has implications on other parts, such as variables where they can be accessed and set via $mf (iirc).\r\n\r\nOn the other hand, I don't think having a variable with different lengths is a good approach, because it can break easily config behaviour when moving from one server to another. It should be same size on 32b or 64b, so if a change to increase the size is to be done, then should be `long long` to ensure it is 64 on all archs. An alternative would be to make the field inside sip_msg_t an array of flag_t values, so can be one or more, even beyond 64b. But this change will have also impact in other parts, therefore would need careful review.\r\n\r\nAs an alternative for now would be using an avp or xavps, and do bitwise operations (`|` and `\u0026`) to set or test."}],"action":{"name":"View Issue","url":"https://github.com/kamailio/kamailio/issues/1288#issuecomment-340394835"}}}</script>