madcowfred / newsmangler

NNTP binary poster with mulitple connection support
37 stars 18 forks source link

asyncNNTP.py add_channel incorrect? #1

Closed alok0 closed 14 years ago

alok0 commented 14 years ago

In more recent versions of python add_channel and del_channel have more parameters, so overriding these methods needs to match them, or else it trips up.

I know I've had this issue with 2.6 and 2.7

But since I am a python noob, I don't know if my fix is correct.

madcowfred commented 14 years ago

I've committed a fix for this but don't have an account I can test posting on right now - it should work though.

alok0 commented 14 years ago

I did one test, seems to work as far as i can tell.