[MERGE] Graph.find_revno()

[Aaron Bentley]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

John Arbash Meinel wrote:
> I'm happy to rename it. It isn't called "find_dotted_revno", and the pure
> "revno" is a static integer that is fixed regardless of branch.

I've never been a big fan of dotted revnos, but you have to admit that
they are referred to as "revnos".

The way I see it is that integer revnos are a way of describing
revisions with respect to a branch.  They are either defined or
undefined for a given revision, depending on the branch.  If the
revision lies in the lefthand ancestry of the head revision, the revno
is defined, and there is only one possible value.  Otherwise, there is
no defined value.

It's true that revnos are fixed in that sense, but revnos are usually
associated with a branch, and I just see potential for less-experienced
bzrlib users to shoot themselves in the foot with this.

> I'll try
> to come
> up with a better name.

Thanks.

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFINZ9o0F+nu1YWqI0RAjyKAJ4xLOE4Lgg/qoFM1QdBhl16r0+EuwCeMcxA
IgLHkZcc+PW4WeJBet3UTBk=
=V5fz
-----END PGP SIGNATURE-----