Aman Shah added the comment:

I had put in the renaming because "ioclass" is not defined in the class itself, 
only in the subclasses. So, instantiating it and using dumps/loads would be 
meaningless and would raise an error. Which is similar to the behavior of an 
abstract class.
Also, the "if(self.f):" part can be improved as you said and I looked at the 
patch for other similar mistakes. But, it's an isolated mistake (at lines 33 
and 74) and so maybe it can be fixed during the merge itself?

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue17299>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to