<p style="white-space: pre-wrap; word-wrap: break-word;">The linter does not like that osmo_str_to_int() sometimes returns a positive errno.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I did that on purpose. Why am I doing this?<br>There is a difference between exceeding the data type range and exceeding the min_val..max_val range.<br>If exceeding the min_val..max_val range, the result will still reflect the exact number parsed.<br>If exceeding the data type range, the result will be clamped to INT_MIN..INT_MAX.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So I thought it is good to distinguish between -ERANGE where our osmo code says "out of range",<br>and strtoll() ERANGE where the library call says "cannot be parsed because there are too many digits".</p><p style="white-space: pre-wrap; word-wrap: break-word;">Does anyone oppose removing the linter V-1?</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25345">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/25345">change 25345</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/c/libosmocore/+/25345"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4dac826aab00bc1780a5258b6b55d34ce7d50c60 </div>
<div style="display:none"> Gerrit-Change-Number: 25345 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 07 Sep 2021 12:01:42 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>