Author rhettinger
Recipients larry, methane, rhettinger, vstinner
Date 2017-01-19.06:48:39
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1484808520.47.0.0842364750678.issue29312@psf.upfronthosting.co.za>
In-reply-to
Content
I like the other AC changes to dict in 29311, but this one seems like it shouldn't be done.  There is too much twisting around existing code to force it to use AC and the benefit will be almost nothing.   dict.update() is mainly used with a list of tuples argument or with another mapping.  The O(1) time spent on the method call is inconsequential compared to the O(n) step of looping over all the inputs and putting them in the dict.  Accordingly, I think this method should be skipped, leaving the current clear, stable, fast-enough code in-place.
History
Date User Action Args
2017-01-19 06:48:40rhettingersetrecipients: + rhettinger, vstinner, larry, methane
2017-01-19 06:48:40rhettingersetmessageid: <1484808520.47.0.0842364750678.issue29312@psf.upfronthosting.co.za>
2017-01-19 06:48:40rhettingerlinkissue29312 messages
2017-01-19 06:48:39rhettingercreate