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 rhettinger
Recipients eric.snow, ezio.melotti, methane, rhettinger, serhiy.storchaka, vstinner
Date 2016-09-22.06:23:35
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1474525415.21.0.833324219916.issue28239@psf.upfronthosting.co.za>
In-reply-to
Content
> I don't suggest to change lru_cach() implementation just now

For now, I would like to have this closed.  It doesn't make sense at the current juncture (with the compact being new, being in flux, and not having guaranteed ordering semantics).

Also, we should have a strong preference for loose coupling and high cohesion.  The lru cache code is principally about tracking recent and should maintain primary responsibility for the action, and the dictionary implementation should be primarily about a clean mapping implementation without having code baggage just for the lru).  

Besides that, there isn't much motivation for change.  The existing code is very good (clear, fast, decoupled, cohensive, and doesn't have compaction issues for cache hits).
History
Date User Action Args
2016-09-22 06:23:35rhettingersetrecipients: + rhettinger, vstinner, ezio.melotti, methane, eric.snow, serhiy.storchaka
2016-09-22 06:23:35rhettingersetmessageid: <1474525415.21.0.833324219916.issue28239@psf.upfronthosting.co.za>
2016-09-22 06:23:35rhettingerlinkissue28239 messages
2016-09-22 06:23:35rhettingercreate