<blockquote>
<p>not get the warning about the sign change on the return values with using -Wall -Wextra only.</p>
</blockquote>
<p>Well, its perfectly legal C to assign <code>int</code> to <code>uint</code>, and there are old time system interfaces that use <code>int</code> where they only return positive values and actually should have used <code>uint</code>, so the warning is spurious (and unfixable) in some cases.  Thats why it is not included in <code>-Wall</code> and <code>-Wextra</code>.</p>
<p>The less used warnings tend to be ones that can cause more unsuppressable spurious results, or just plain stupid results (see a recent discussion between <a href="https://github.com/b4n" class="user-mention">@b4n</a> and I about returning small structs from functions that was triggered by one of his warnings <g-emoji alias="wink" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f609.png" ios-version="6.0">😉</g-emoji> ).  Thats why <a href="https://github.com/b4n" class="user-mention">@b4n</a> doesn't advise his option set and use it on Travis, the results cannot be taken at face value, they need interpretation.</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/geany/geany-plugins/pull/598#issuecomment-323883254">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ5RBbrsMbsK9teKzzydaEKCrzjRQks5sahulgaJpZM4O391f">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJzUeD0Mn0wxvol9j6fgiQpS2OkZ1ks5sahulgaJpZM4O391f.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/geany/geany-plugins/pull/598#issuecomment-323883254"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request 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/geany/geany-plugins","title":"geany/geany-plugins","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/geany/geany-plugins"}},"updates":{"snippets":[{"icon":"PERSON","message":"@elextr in #598: \u003e  not get the warning about the sign change on the return values with using -Wall -Wextra only.\r\n\r\nWell, its perfectly legal C to assign `int` to `uint`, and there are old time system interfaces that use `int` where they only return positive values and actually should have used `uint`, so the warning is spurious (and unfixable) in some cases.  Thats why it is not included in `-Wall` and `-Wextra`.\r\n\r\nThe less used warnings tend to be ones that can cause more unsuppressable spurious results, or just plain stupid results (see a recent discussion between @b4n and I about returning small structs from functions that was triggered by one of his warnings :wink: ).  Thats why @b4n doesn't advise his option set and use it on Travis, the results cannot be taken at face value, they need interpretation."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany-plugins/pull/598#issuecomment-323883254"}}}</script>