[PATCH 01 of 10] bundlerepo: fix outdated comment
Benoit Boissinot
bboissin at gmail.com
Wed Jan 16 14:11:31 CST 2013
LGTM
On Wed, Jan 16, 2013 at 8:57 PM, Mads Kiilerich <mads at kiilerich.com> wrote:
> # HG changeset patch
> # User Mads Kiilerich <madski at unity3d.com>
> # Date 1358365292 -3600
> # Node ID 528c8be7d397d7c2638858006bdd9d55e3e2c314
> # Parent e3f5cef11d6a8eafdbaefa9b6c3d0e91fc3273fc
> bundlerepo: fix outdated comment
>
> Comment was made invalid by 01ee43dda681.
>
> diff --git a/mercurial/bundlerepo.py b/mercurial/bundlerepo.py
> --- a/mercurial/bundlerepo.py
> +++ b/mercurial/bundlerepo.py
> @@ -20,14 +20,14 @@
> class bundlerevlog(revlog.revlog):
> def __init__(self, opener, indexfile, bundle, linkmapper):
> # How it works:
> - # to retrieve a revision, we need to know the offset of
> + # basemap which keeps track of which revision the delta of each
> bundle
> + # revision apply to.
>
This part looks a bit weird though, maybe make it a proper sentence?
> + #
> + # To retrieve a revision, we need to know the offset of
> # the revision in the bundle (an unbundle object).
> - #
> - # We store this offset in the index (start), to differentiate a
> - # rev in the bundle and from a rev in the revlog, we check
> - # len(index[r]). If the tuple is bigger than 7, it is a bundle
> - # (it is bigger since we store the node to which the delta is)
> - #
> + # We store this offset in the index (start). To differentiate a
> + # rev in the bundle from a rev in the revlog, we check revision
> + # against basemap.
> opener = scmutil.readonlyvfs(opener)
> revlog.revlog.__init__(self, opener, indexfile)
> self.bundle = bundle
> @@ -385,4 +385,3 @@
> other.close()
>
> return (localrepo, csets, cleanup)
> -
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20130116/665370a7/attachment.html>
More information about the Mercurial-devel
mailing list