Josh Rosenberg added the comment:

So, incredibly minor note:

This will prevent a ridiculous use case of passing in a mutable object as the 
argument (say, a list), and mutating it between truthy and falsy values 
(appending or clearing) to toggle behaviors in an existing Encoder.

Note: As stated, this is a ridiculous use case, and I don't think we should be 
handcuffed by compatibility with an insane behavior that no one has likely ever 
used. The docs never guarantee that those values are examined live, so it's an 
implementation detail, and not a useful one at that. I just want to mention it 
so there is a small note in MISC/News or the like to document that slight tweak 
in observed behavior.

Otherwise, change looks good to me.

----------
nosy: +josh.r

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

Reply via email to