[Svnmerge] [PATCH] Bidirectional merging patch for svnmerge.py
Blair Zajac
blair at orcaware.com
Thu Feb 23 15:13:38 PST 2006
Raman Gupta wrote:
> @@ -330,6 +331,9 @@
>
> def __iter__(self):
> return iter(self.sorted())
> +
> + def __len__(self):
> + return len(self._revs.keys())
Hi Raman,
I'm working through the patch to see what to merge into svn.collab.net.
I'm not too fond of the 'len' function operating on a RevisionList object, as
the name isn't too clear. Before I looked at the code, I was wondering if the
length is the difference between the minimum and the maximum revision, but see
that it's the count of revisions.
I would prefer to use a name such as count for this.
Comments?
Also, out of curiosity, do you have any code that uses this? It doesn't appear
that the bidirectional patch uses this new code.
Regads,
Blair
More information about the Svnmerge
mailing list