[PATCH] Use merge3 for text merging
Martin Pool
mbp at sourcefrog.net
Tue Jul 26 19:11:31 BST 2005
On 26 Jul 2005, Aaron Bentley <aaron.bentley at utoronto.ca> wrote:
> These patches implement
> 1. the merge_factory concept, so that different merge approaches can be used
> 2. the Merge3 merge, used by default
Good stuff.
> I needed a way to determine whether the merge produced conflicts. I
> didn't want just scan for '<<<<<<<' because that might already be
> present in the original file-- i.e. not an indicator of a *new*
> conflict. So I set the start marker to an unlikely string, and replaced
> it with '<<<<<<' when I encountered it. It's unlikely to cause
> real-world problems, but I'd rather be able to directly determine
> whether any conflicts ocurred.
I think merge3 should change to avoid the need for the ugly hack. It
can do either of two things:
- just simply tell the caller whether there were conflicts or not
- rather than inserting the conflict markers itself (which is a bit
ugly) instead return a data structure describing conflicted and
unconflicted regions
I as going to do the second but piked until it was needed. Whch do
you want?
--
Martin
More information about the bazaar
mailing list