[PATCH] avoid false positives for addremove -s, mark II

Erling Ellingsen erlingalf at gmail.com
Sun Feb 18 13:53:12 CST 2007


Polished version of the previous patch.

> This looks mostly ok, but gmail has the annoying tendency to mangle
> inline patches.  Can you send it again as an attachment?

Really? Neither 'patch' nor 'hg import' complained at my end. Well, attached.

> Put the import at the top of the file, with the other imports

I wanted to avoid loading an extra file for the 1-ε cases where it's
not used. However, I see that bdiff is included by mdiff anyway, so it
didn't help. Moved.

> This assert shouldn't be needed.  Well, asserts obviously should never
> trigger, but if we really want that check, there are probably better
> places for it.

I put the assert there as I expected the return value of bdiff to
change in the near future (given that it is returning uninitialized
bits of memory right now). Removed.

-- 
 Erling
-------------- next part --------------
A non-text attachment was scrubbed...
Name: similar.patch
Type: application/octet-stream
Size: 4033 bytes
Desc: not available
Url : http://www.selenic.com/pipermail/mercurial-devel/attachments/20070218/1c863482/similar.obj


More information about the Mercurial-devel mailing list