Raymond Hettinger added the comment:

The core idea is reasonable.  

The patch overreaches by adding readonly warnings to __setitem__ and 
__delitem__ which will spew-out on every call (these could be ignored or set to 
warn once by the user but that is PITA).

Minor nit.  We have a little PEP-8 overkill on the string splits:

+            with self.assertWarnsRegex(DeprecationWarning,
+                                       "The database file is missing, the "
+                                       "semantics of the 'c' flag will "
+                                       "be used."):

----------

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

Reply via email to