Nick Coghlan added the comment:

Aye, suppress is only intended for use around a single line of code. Using it 
for an entire function would be "OnError Resume Next" levels of poor coding 
style.

I'm also -1 on implicitly wrapping redirect_stdout around functions due to the 
immediate thread safety problem doing so introduces. The stdout redirection 
really only makes sense in a single-threaded scripting context, and using it as 
a decorator rather than inline makes it far too easy to inadvertently violate 
that constraint.

As far as the "explicitly inherit from object or not" goes, it wouldn't 
surprise me if that's just a "migrated from Python 2" vs "first introduced in 
Python 3" distinction. It's not something I would change solely for the sake of 
consistency.

----------
resolution:  -> rejected
stage:  -> resolved
status: open -> closed

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

Reply via email to