This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author neologix
Recipients giampaolo.rodola, neologix, vstinner, yselivanov
Date 2017-04-07.22:41:25
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CAH_1eM3MzwGSCxpMukYA2AKNhC=U85k3Ly7KExeFPJfiaMVRJA@mail.gmail.com>
In-reply-to <1491593128.93.0.93926480117.issue30014@psf.upfronthosting.co.za>
Content
This refactoring was already suggested a long time ago, and at the
time both Guido and I didn't like it because it makes the code
actually more complicated: DRY in this case doesn't apply IMO.

Also, this whole thread is a repeat of:
http://bugs.python.org/issue18932

At the time, I already asked for one realistic use case demonstrating
the benefit of implementing modify() natively instead of
unregister()+register().
I know that this code makes modify() faster, but as I said multiple
times, I'd like to se the impact on something else than a
micro-benchmark: arguably if you propose this it's because you've
profiled/found it to be an actual bottleneck, do you have *one*
benchmark to support this change?
History
Date User Action Args
2017-04-07 22:41:26neologixsetrecipients: + neologix, vstinner, giampaolo.rodola, yselivanov
2017-04-07 22:41:26neologixlinkissue30014 messages
2017-04-07 22:41:25neologixcreate