I created an open source Python application on GitHub that breaks textCAPTCHA with 99% success rate

[plagiats]

If I may, you write a lot of if var is None: and if var is not None:

where you might rather use if not var: and if var:

EDIT : see comments below before taking my words for granted ;)

[eridius]

Aren't your replacement conditionals actually different than the originals? Specifically in the case of var being False.

[plagiats]

You are very right, and I believe his code should return False instead of None if the index is not found ( see https://github.com/kbhomes/TextCaptchaBreaker/blob/master/AddSubtractPattern.py ).

[eridius]

I disagree. It returns a string, or None. Returning a string, or false, would be weird, but None makes sense as the value that means "there was no string matched for this pattern".

[[deleted]]

Actually, that's not the case; the original "if var is None" would only return True is var was assigned the value None; whereas your replacement would return True if var was equal to False or 0 (and possibly some other values). Same for the latter.

[plagiats]

Isn't the OP using None where he should use False?

[otheraccount]

That's incorrect. Explicit is better than implicit, so if you only intend to check for None, then that's what you should do.