On 02/18/2017 04:51 AM, Jun Wu wrote:
Excerpts from Stanislau Hlebik's message of 2017-02-17 16:06:33 +0000:
This is implementation of two caches (nonpublic + precursor) using
serialized sorted lists and sets
https://bitbucket.org/stashlebik/hg/commits/99879579ac2848a2567810b677d8344150a7b319?at=hiddenbitmaps_lists
I had a quick look. Here are my suggestions:
1. Prefer caching at a lower-level, so they get more widely used.
+1 on that,
Practically, do not change localrepo.py, change phasecache.py and
obsolete.py instead.
I happened to do some refactoring in phasecache [1]. I'll send a V2 soon,
since the change seems related.
Note: phasecache is some C code so you need some data to prove the change
is worthy.
2. The "Python set" serialization is already in repoview.py. Reuse it.
Maybe move "_writehiddencache" to scmutil, or a new simple class.
"tryreadcache" could be changed and then moved.
Then the functions could be reused in obsolete.getrevs.
[1]:
https://www.mercurial-scm.org/pipermail/mercurial-devel/2017-February/092817.html
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
--
Pierre-Yves David
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel