Serhiy Storchaka added the comment: The idea looks interesting and the code looks correct. But the patch decreases readability and I'm not sure, that the benefit is worth it. Timings looks good at first glance, but frozenset hash is cached and is calculated only once per instance. I afraid that the benefit of hash calculation optimization is dwarfed by the time of frozenset creation itself.
If other core developer supports this idea I have no strong objections. See my comments on Rietveld. ---------- assignee: serhiy.storchaka -> _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue24762> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
