<p>I just noticed my previous tests were wrong:<br>
the warnings are not just warnings, actually grep doesn't find matches in sub directories without the patch. With the patch, the warnings are gone and matches will be found properly.</p>

<p>So, we probably should apply it.</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/issues/1229#issuecomment-247847635">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJz0n49SES_NjG7Gp85gHtucYiyMRks5qrTwLgaJpZM4J6MLJ">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJxnhh3NwAAg5atNUDf9oYI-W-sSPks5qrTwLgaJpZM4J6MLJ.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/issues/1229#issuecomment-247847635"></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/geany/geany","title":"geany/geany","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"}},"updates":{"snippets":[{"icon":"PERSON","message":"@eht16 in #1229: I just noticed my previous tests were wrong:\r\nthe warnings are not just warnings, actually grep doesn't find matches in sub directories without the patch. With the patch, the warnings are gone and matches will be found properly.\r\n\r\nSo, we probably should apply it."}],"action":{"name":"View Issue","url":"https://github.com/geany/geany/issues/1229#issuecomment-247847635"}}}</script>