murgo / IrssiNotifier

notifies android phones from irc private messages
Apache License 2.0
171 stars 63 forks source link

irssinotifier.pl: move sending to its own thread #122

Closed qvr closed 11 years ago

qvr commented 11 years ago

Even with the wget timeouts irssi will still block for a while during network outages. Move sending logic inside its own thread to avoid this.

This closes issue #27

murgo commented 11 years ago

Thanks!

This is something that I've been meaning to do for a long time, but frankly I suck at perl and haven't really scripted Irssi before, so... I'll test and merge this some day when I have the time.

I'm not super happy for this to skip messages though. If many messages arrive right after each other (say, you paste 2 lines and wish to get them to your phone), I'd want it to have some kind of queue (with max limit of course, let's say 10) there.

I can of course implement that myself too.

murgo commented 11 years ago

Looks good. I'll test this for a bit before actually "releasing" it.

Thanks again, this really was needed.