[PATCH 0 of 6] bookmarks: propertycache and cleanups

David Soria Parra sn_ at gmx.net
Mon Dec 21 05:37:05 CST 2009


On 2009-12-21, Nicolas Dumazet <nicdumz at gmail.com> wrote:
> Hello!
>
> Bookmarks is caching manually _bookmarkcurrent and _bookmarks. For clarity, I
> find it much more easier to use propertycache().
> The code is simplified and easier to read.
>
> I am not sure of how much of the namespace of localrepo is allowed to be
> cluttered by extension helper functions. If we prefer keeping the logic
> outside of repo subclasses, the last patch can be dropped from this serie.

This series looks fine for me, I think it should go into crew.



More information about the Mercurial-devel mailing list