Giampaolo Rodola' <g.rod...@gmail.com> added the comment: The patch as-is can't be accepted if not for Python 4.x maybe, obviously because it's just too breaking.
A proper patch would provide aliases for the removed attributes and raise a DeprecationWarning in case they are accessed. It would be suitable for the the next major release (3.2 at the current time) and the total removal of the deprecated attributes would happen only in 3.3 if not later. Also, I see no reason in turning public attributes like self.__line, self.__state and self.__greeting. They should just be private (one underscore) and their name kept the same (e.g. no __server -> smtp_server, __greeting -> seen_greting, etc...). Finally, tests should definitively be written. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue4184> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
