<blockquote>
<p>It almost works, but I had to tweak the code:</p>
</blockquote>
<p>Ok, thanks. Looks good, I didn't realize pcregrep doesn't have an -E option.</p>
<blockquote>
<p>The only other remark is the exclude-dir and exclude and include syntax works via regexps not globbing</p>
</blockquote>
<p>Hmm, it's frustrating that they repurposed those options, it's so close to being a drop in replacement. (They could have added regexp file matching with different option names). We could just disable the file filter row when pcregrep is set.</p>
<blockquote>
<p>As for whether we should use grep...<br>
I don't see why it would be hard to do this internally though, given that Geany already implements regexp searching of files</p>
</blockquote>
<p>It doesn't. Geany loads files into scintilla. It would be simple to implement trivial file searching, but grep and pcregrep have many optimisations vs simple linear search and reading entire files into buffers.</p>
<blockquote>
<p>Possibly people are using some exotic options</p>
</blockquote>
<p>Yes, I expect we would struggle with this. For example, pcregrep has an option to treat lone CR or LF as a new line.</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/2322?email_source=notifications&email_token=AAIOWJ2Y6SUM6W4W54XQMRTQORFFJA5CNFSM4I2FFBWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBEEPZQ#issuecomment-541607910">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ4VTGEKOONJBVNK6WTQORFFJANCNFSM4I2FFBWA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAIOWJ5JGASPLIQQ2OBN533QORFFJA5CNFSM4I2FFBWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBEEPZQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/issues/2322?email_source=notifications\u0026email_token=AAIOWJ2Y6SUM6W4W54XQMRTQORFFJA5CNFSM4I2FFBWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBEEPZQ#issuecomment-541607910",
"url": "https://github.com/geany/geany/issues/2322?email_source=notifications\u0026email_token=AAIOWJ2Y6SUM6W4W54XQMRTQORFFJA5CNFSM4I2FFBWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBEEPZQ#issuecomment-541607910",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>