[PATCH 1 of 3 stable] bookmarks: remove bogus nullmerge check in updatebookmarks

Matt Mackall mpm at selenic.com
Wed Jan 30 23:48:34 CST 2013


On Wed, 2013-01-30 at 18:16 -0800, Siddharth Agarwal wrote:
> # HG changeset patch
> # User Siddharth Agarwal <sid0 at fb.com>
> # Date 1359596994 28800
> # Branch stable
> # Node ID 63c49a4df508562b8fd91a5da1b03b9fbadf08f8
> # Parent  c795c9f87792fef98b358ae88f90c4003e9bbe4d
> bookmarks: remove bogus nullmerge check in updatebookmarks
> 
> nstate[v] is a node, not an int, and the nullmerge check was done while
> building nstate anyway.

I've queued these three for stable, thanks.

In the future, drive-by cleanups are actually not wanted in stable.
There's no upside to this sort of patch in terms of bugs fixed, user
experience improved, etc., there are only opportunities to accidentally
break things.

While this change is "obviously correct", every programmer has also had
the experience of spending hours trying to find a bug, only to discover
it was an invisible one-character typo. Thus every programmer should
also know that "obviously correct" and "correct" are not quite the same.

> diff --git a/hgext/rebase.py b/hgext/rebase.py
> --- a/hgext/rebase.py
> +++ b/hgext/rebase.py
> @@ -498,9 +498,8 @@ def updatebookmarks(repo, nstate, origin
>      marks = repo._bookmarks
>      for k, v in originalbookmarks.iteritems():
>          if v in nstate:
> -            if nstate[v] > nullmerge:
> -                # update the bookmarks for revs that have moved
> -                marks[k] = nstate[v]
> +            # update the bookmarks for revs that have moved
> +            marks[k] = nstate[v]
>  
>      marks.write()
>  
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel


-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list