search

tvkanters / Dopestreamer

A wrapper for Livestreamer that offers a GUI to easily launch streams.
10 stars 2 forks source link

Look into RAM usage #18

Closed tvkanters closed 10 years ago

tvkanters commented 10 years ago

Dopestreamer is using ~90MB RAM. This should be reduced.

VackerSimon commented 10 years ago

Dopestreamer is using 100MB+ RAM, 347MB reported. This needs to be reduced.

tvkanters commented 10 years ago

I believe this is only fixable by switching away from JavaFX. Through VisualVM and online resources, it seems pretty obvious that JavaFX simply has a large memory footprint. As an example, comparisons like http://etfdevlab.blogspot.nl/2011/06/javafx-vs-java-swing-vs-awt.html show that JavaFX has twice the memory consumption of AWT.

The 350MB spike is likely due to the monitor tools, as using those is the only way I managed to get the memory usage above 100MB. (Especially monitor tools multiple times makes the memory jump.)

I don't believe that the slightly high memory usage is worth changing the entire GUI to AWT considering the effort and the risk of ruining the visuals. I'd vote to close the issue, unless you want to look further into is, Xphome.

VackerSimon commented 10 years ago

Yeah