cdv test failures

James Henstridge james.henstridge at gmail.com
Fri May 26 07:09:52 BST 2006


On 26/05/06, John Arbash Meinel <john at arbash-meinel.com> wrote:
> Martin Pool wrote:
> > Is it even possible to override something called __helper?  I would have
> > thought mangling meant it always called the original implementation.
> >
> > +1 for this patch.  One comment - since you're overriding it, perhaps
> > you should use a better name than __helper -- even just
> > _find_matching_blocks.
> >
>
> I was surprised to see it work as well. But it definitely did. I always
> thought that with the mangling the parent would always call its own
> function.

It worked because both difflib.SequenceMatcher and
bzrlib.patiencediff.SequenceMatcher rewrote __helper to
_SequenceMatcher__helper().  If the classes had different names, it
would not have worked.

James.




More information about the bazaar mailing list