<?xml version="1.0" encoding="UTF-8"?><rss xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:atom="http://www.w3.org/2005/Atom" version="2.0"><channel><title><![CDATA[Improving Find in Files performance]]></title><description><![CDATA[<p dir="auto">Hi Notepad++ team!</p>
<p dir="auto">I’ve created a pull request that improves the Find in Files performance by re-using the existing approach with several optimizations, and thus making it 2…3 times faster.</p>
<p dir="auto"><a href="https://github.com/notepad-plus-plus/notepad-plus-plus/pull/18376" rel="nofollow ugc">https://github.com/notepad-plus-plus/notepad-plus-plus/pull/18376</a></p>
<p dir="auto">I haven’t invented anything new, just optimized what was already there.<br />
I realize the code review may create an impression that several parts have been completely re-written, but that’s no true.<br />
In fact, the <em>existing</em> code has been refactored by extracting parts of several functions into separate helpers, so these moved blocks of code together with new helpers create an impression of a lot of changes. But if you follow the code logic, or just manually copy-paste the separate helpers into the places where these helpers are called, you’d realize the code flow remains the same.<br />
Please feel free to review and test.<br />
I’m writing here because I’m afraid the longer this pull request remains open, the higher are chances of merge conflicts to appear in the affected files.</p>
]]></description><link>https://community.notepad-plus-plus.org/topic/27663/improving-find-in-files-performance</link><generator>RSS for Node</generator><lastBuildDate>Thu, 24 Sep 2026 10:00:39 GMT</lastBuildDate><atom:link href="https://community.notepad-plus-plus.org/topic/27663.rss" rel="self" type="application/rss+xml"/><pubDate>Tue, 22 Sep 2026 12:30:14 GMT</pubDate><ttl>60</ttl><item><title><![CDATA[Reply to Improving Find in Files performance on Wed, 23 Sep 2026 22:13:24 GMT]]></title><description><![CDATA[<p dir="auto"><a class="plugin-mentions-user plugin-mentions-a" href="/user/vitalii-dovgan" aria-label="Profile: Vitalii-Dovgan">@<bdi>Vitalii-Dovgan</bdi></a> ,</p>
<p dir="auto">Even though you just “moved” stuff, it’s a lot of lines of code changed, from the <code>diff</code> perspective, and Don manually reviews all of that.  So it’s not an easy one to review.</p>
<p dir="auto">And since you didn’t attach the PR to any open Issue (though I’m sure there must be more than one asking for improved FiF performance), it’s probably a low priority in his mind (and I’m surprised he didn’t reject it out of hand.  It might be because it came from you that it wasn’t rejected).</p>
<p dir="auto">I recommend searching out one or more Issues to attach it to.</p>
<p dir="auto">But yes, I do hope it’s able to be merged before there is a conflict.</p>
]]></description><link>https://community.notepad-plus-plus.org/post/106382</link><guid isPermaLink="true">https://community.notepad-plus-plus.org/post/106382</guid><dc:creator><![CDATA[PeterJones]]></dc:creator><pubDate>Wed, 23 Sep 2026 22:13:24 GMT</pubDate></item></channel></rss>