Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible EventEmitter memory leak detected. 11 did-stop-loading listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit (Use electron --trace-warnings ... to show where the warning was created) #107

Open
raphael10-collab opened this issue Oct 29, 2022 · 5 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@raphael10-collab
Copy link

From third tab onwards:

raphy@pc:~/raj-browser$ yarn start
vite v3.2.0 building for production...
✓ 173 modules transformed.
out/main/index.js 0.39 KiB
out/main/multipart-parser.a8ad5d2c.js 10.10 KiB
out/main/index.76de7e3c.js 774.74 KiB
vite v3.2.0 building for production...
✓ 1 modules transformed.
out/preload/index.js 0.41 KiB
polyfillModulePreload is deprecated. Use modulePreload.polyfill instead.
vite v3.2.0 building for production...
✓ 16 modules transformed.
✓ [webfont-dl] 0 css downloaded.
✓ [webfont-dl] 0 fonts downloaded.
../../out/renderer/assets/arrow_left.11420a4f.svg 0.28 KiB
../../out/renderer/assets/arrow_right.86451b58.svg 0.27 KiB
../../out/renderer/assets/add.fb371332.svg 0.37 KiB
../../out/renderer/assets/dismiss.36817ba6.svg 0.40 KiB
../../out/renderer/assets/arrow_clockwise.08941c54.svg 0.35 KiB
../../out/renderer/index.html 0.31 KiB
../../out/renderer/assets/index.3f873000.js 31.62 KiB
../../out/renderer/assets/index.2b4183e5.css 3.14 KiB

start electron app...
[8964:1029/143414.757408:ERROR:gpu_memory_buffer_support_x11.cc(44)] dri3 extension not supported.

new-tab https://google.com

close-tab sk6hj1kvy

new-tab https://google.com

new-tab https://google.com
 (x2)
(node:8934) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 did-stop-loading listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
(Use `electron --trace-warnings ...` to show where the warning was created)

new-tab https://google.com

(node:8934) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 did-stop-loading listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit

new-tab https://google.com

(node:8934) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 did-stop-loading listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit

new-tab https://google.com

(node:8934) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 did-stop-loading listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
@Rajaniraiyn
Copy link
Owner

Yeah I am aware if this warning.
This is is unavoidable now because the browser is entirely on event emitter and event listeners to ensure IPC between main and renderer process.
I haven't increased the limit manually as I'm looking on any alternative ways to achieve IPC rather than the events.

If you had any alternate ways then please suggest here it will be helpful for the further development of this browser.

@Rajaniraiyn Rajaniraiyn self-assigned this Oct 31, 2022
@Rajaniraiyn Rajaniraiyn added the help wanted Extra attention is needed label Oct 31, 2022
@raphael10-collab
Copy link
Author

@Rajaniraiyn may be this usage of node-ipc can be source of inspiration: https://github.com/thanhlmm/electron-multiple-tabs

@Rajaniraiyn
Copy link
Owner

Thank you @raphael10-collab ill have a look into it

@raphael10-collab
Copy link
Author

@Rajaniraiyn
Copy link
Owner

@raphael10-collab Due to some other projects I couldn't have time to implement a different approach. if you saw anything effective, please open a pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants