Terry J. Reedy <tjre...@udel.edu> added the comment:
Your code is buggy because 'cursor' is not a dict or anything like one.
On the other hand, adding retval to capture and return the return from
Widget.configure looks correct. (And it will not prevent kw.update('cursor')
from raising. The current code is
def configure(self, cnf=None, **kw):
"""Modify or query scale options.
Setting a value for any of the "from", "from_" or "to" options
generates a <<RangeChanged>> event."""
if cnf:
kw.update(cnf)
Widget.configure(self, **kw)
if any(['from' in kw, 'from_' in kw, 'to' in kw]):
self.event_generate('<<RangeChanged>>')
)
To review, test, and make the change, a github Pull Request (PR) is needed. I
will make one.
----------
nosy: +serhiy.storchaka, terry.reedy
_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue39152>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
