cerebroapp / cerebro

🔵 Cerebro is an open-source launcher to improve your productivity and efficiency
https://www.cerebroapp.com/
MIT License
8.27k stars 454 forks source link

fix: memory leak on reloading #631

Closed dubisdev closed 1 year ago

dubisdev commented 1 year ago

We were reloading only the two renderer processes. This was causing the main process to continue subscribed to older renderers that were not existing.

(node:22748) MaxListenersExceededWarning: Possible EventEmitter memory leak detected.
11 render-view-deleted listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit

Now when using the reload command, both the main process and the renderer processes will be reloaded. This makes Cerebro to reload a bit slower but avoids memory leaks.