MooseElkingtons / mutinyirc

extensible IRC plugin with a simple, yet elegant design.
MIT License
0 stars 1 forks source link

IRCOutputThread idles 100% CPU #1

Closed upwhere closed 10 years ago

upwhere commented 10 years ago

Your polling method bypasses the interval, making it poll the queue at 100% CPU. I'm amazed to see something like this here, surely this has been noticed before?

For an easy fix you move the Thread.sleep() out the conditional block, though it'd be better if you forfeit polling and use proper synchronisation.

MooseElkingtons commented 10 years ago

It was originally like this, but I had complaints about latency issues. I'll look into a better fix.

MutinyCraft commented 10 years ago

i cry evry tiem