Thread: [BUG-FIX]Hyperion's Event Manager - used in PVP Planet source too

Not everyone saw Graham's edit, so I decided to create another thread.

Originally Posted by Graham

Important notice to all Hyperion users! There is a memory leak currently present which has not been fixed in any official releases or in the Subversion repository. To fix it, open the EventManager class and after the isRunning() check, add an else which just returns from the function. If you do not fix this, your server will gradually consume more memory and CPU power and some actions may continue forever. (NOTE: this is not related to Sir Harry's memory problem - one of the first things I did was to ask him to apply this fix which did not help. According to Sir Sean, it isn't a problem with MINA either - it was something he added - which is kind of what I expected anyway since I had not experienced it during testing Hyperion and the Rune-Server official server.)

Spoonfeed:
In your EventManager.java search for:

Code:

if(event.isRunning()) {
	event.execute();
}

As Graham said, all you need to do is just to add a return, so the code will be:

Code:

if(event.isRunning()) {
	event.execute();
} else {
	return;
}

NOTE: Hyperion's EventManager is also used in [Only registered and activated users can see links. ].