search

cerebroapp / cerebro-basic-apps

📦 Cerebro plugin for applications search and launch on windows and linux
MIT License
11 stars 11 forks source link

Let's not call `toString()` on each key press during search #7

Closed nazar-pc closed 6 years ago

nazar-pc commented 6 years ago

toString() results do not change between searches, let's compute them once and store permanently. This should radically improve situation with https://github.com/KELiON/cerebro/issues/253.

nazar-pc commented 6 years ago

Good, I've closed that PR and removed TODO comment

nazar-pc commented 6 years ago

Would be nice to see fresh Cerebro build soon, according to Dev Tools profiling this should be a major win not only for https://github.com/KELiON/cerebro/issues/253, but for responsiveness in general.

KELiON commented 6 years ago

@nazar-pc thanks! I've just published new version of plugin with your changes.