refresh: 0 should disable refreshing

sunaku / wmiirc

Ruby configuration for WMII window manager

ISC License

82 stars 26 forks source link

refresh: 0 should disable refreshing #11

Closed nougad closed 14 years ago

nougad commented 14 years ago

In your comment1 you suggest a refresh rate of 0. But at the moment a refresh rate of 0 will course 100% CPU load because the refresh threads loops with sleep 0.

See my fix here2.

sunaku commented 14 years ago

Good find. I thought that the thread would simply sleep forever when given refresh: 0 and you could programatically call Thread#wakeup on it to repaint the status bar applet.

sunaku commented 14 years ago

Solved in sunaku/wmiirc@6580d40afb3e0f896bf9320903f052ee06208263 by setting "refresh" to nil.

nougad commented 14 years ago

works well and is prettier than my hack. Thank you