<p> Attention is currently required from: George Joseph. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/16621">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/16621?tab=comments">Patch Set #4:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Just curious... […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">In patchsets 1 throgh 3 PRId64 was used. In patchset 4 it wouldn't make sense as the patch tries to avoid these platform details.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/16621?tab=comments">Patch Set #4:</a> </p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't know why apparently you didn't see my reply. I hope this time you do.</p><p style="white-space: pre-wrap; word-wrap: break-word;">PRId64 is fine for print functions. But it doesn't help you when scanning. Then you have to make your own assumptions, checking if the platform you're (cross-)compiling for uses (long) or (long long). There may even portability issues with PRId64 itself, because maybe there are libcs (old ones?) that don't define it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">A lot of guesswork involved. Today that is, not just "some time in the future", when more libcs will make changes to address the year 2038 issue. For instance, support for target "Arc" was added to glibc a while ago. That's a 32 bit target, but glibc sets it up with a 64 bit time_t. I don't know why. But I'm venturing a guess that on Arc targets you'd run into problems right now with glibc because of it. Asterisk will loose the AOR as soon as a client registers it. Which is what I saw happening on a 32 bit mips target with musl 1.2.0.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So always using (long long) seems to be the way with the least amount of effort required, while providing maximum portability, in my opinion.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This isn't my idea, even. A while ago I asked on the musl mailing list and got good feedback there. The person was even nice enough to send a pull request to freeswitch: https://github.com/signalwire/freeswitch/pull/1409</p><p style="white-space: pre-wrap; word-wrap: break-word;">I also sent a pull request to kamailio, which got accepted: https://github.com/kamailio/kamailio/pull/2894</p><p style="white-space: pre-wrap; word-wrap: break-word;">I really hope you see my reply this time, because it took me at least 10 minutes to write it 😊</p><p style="white-space: pre-wrap; word-wrap: break-word;">Kind regards,<br>Seb<br></p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/16621">change 16621</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/16621"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ic8d61b26033f5c486b917e738c9608b0923a844e </div>
<div style="display:none"> Gerrit-Change-Number: 16621 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Sebastian Kemper <sebastian_ml@gmx.net> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-CC: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Attention: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 16 Nov 2021 21:44:56 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>