FAForever / ice-adapter

Deprecated C++ ICE adapter - please use java-ice-adapter instead!
https://github.com/FAForever/java-ice-adapter
GNU General Public License v3.0
11 stars 10 forks source link

node.exe in taskmanager increasing memory usage after few games #38

Closed Nojoke185 closed 7 years ago

Nojoke185 commented 7 years ago

Observed after the setons game that node.exe was using around 75000 (ish) kb of Ram Prior to start of open palms game it started to constantly climb by about 4k per sec At next game start (open palms) it seemed to stabilise and hold around 79mb

ice-adapter.txt

Following palms game it was using 84332k and stable Earlier in the day node had been using around 30mb

May be ok, logging for review if required

muellni commented 7 years ago

85 MB is way too much but I think nodejs' and webrtcs internal memory management is out of my reach. I will reopen this if node is indeed eating up users memory.