/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/merge3.py

  • Committer: John Arbash Meinel
  • Date: 2008-03-04 14:25:46 UTC
  • mto: This revision was merged to the branch mainline in revision 3279.
  • Revision ID: john@arbash-meinel.com-20080304142546-zuwwy0o9roo14928
Implement cherrypick support for Merge3
When merging a cherrypick, use a slightly different resolve logic.
When encountering a conflict, the new logic does not include lines that
were present in BASE that are conflicting with OTHER.
This is done since a cherrypick is (by definition) avoiding changes that
are present in the base.
(related to bug #151731)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/merge3.py
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2004, 2005 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
18
18
# mbp: "you know that thing where cvs gives you conflict markers?"
19
19
# s: "i hate that."
20
20
 
21
 
from bzrlib import (
22
 
    errors,
23
 
    patiencediff,
24
 
    textfile,
25
 
    )
 
21
 
 
22
from bzrlib.errors import CantReprocessAndShowBase
 
23
import bzrlib.patiencediff
 
24
from bzrlib.textfile import check_text_lines
26
25
 
27
26
 
28
27
def intersect(ra, rb):
37
36
    >>> intersect((0, 9), (7, 15))
38
37
    (7, 9)
39
38
    """
40
 
    # preconditions: (ra[0] <= ra[1]) and (rb[0] <= rb[1])
41
 
 
 
39
    assert ra[0] <= ra[1]
 
40
    assert rb[0] <= rb[1]
 
41
    
42
42
    sa = max(ra[0], rb[0])
43
43
    sb = min(ra[1], rb[1])
44
44
    if sa < sb:
57
57
            return False
58
58
    else:
59
59
        return True
60
 
 
 
60
        
61
61
 
62
62
 
63
63
 
67
67
    Given BASE, OTHER, THIS, tries to produce a combined text
68
68
    incorporating the changes from both BASE->OTHER and BASE->THIS.
69
69
    All three will typically be sequences of lines."""
70
 
 
71
 
    def __init__(self, base, a, b, is_cherrypick=False, allow_objects=False):
72
 
        """Constructor.
73
 
 
74
 
        :param base: lines in BASE
75
 
        :param a: lines in A
76
 
        :param b: lines in B
77
 
        :param is_cherrypick: flag indicating if this merge is a cherrypick.
78
 
            When cherrypicking b => a, matches with b and base do not conflict.
79
 
        :param allow_objects: if True, do not require that base, a and b are
80
 
            plain Python strs.  Also prevents BinaryFile from being raised.
81
 
            Lines can be any sequence of comparable and hashable Python
82
 
            objects.
83
 
        """
84
 
        if not allow_objects:
85
 
            textfile.check_text_lines(base)
86
 
            textfile.check_text_lines(a)
87
 
            textfile.check_text_lines(b)
 
70
    def __init__(self, base, a, b, is_cherrypick=False):
 
71
        check_text_lines(base)
 
72
        check_text_lines(a)
 
73
        check_text_lines(b)
88
74
        self.base = base
89
75
        self.a = a
90
76
        self.b = b
108
94
            elif self.a[0].endswith('\r'):
109
95
                newline = '\r'
110
96
        if base_marker and reprocess:
111
 
            raise errors.CantReprocessAndShowBase()
 
97
            raise CantReprocessAndShowBase()
112
98
        if name_a:
113
99
            start_marker = start_marker + ' ' + name_a
114
100
        if name_b:
147
133
    def merge_annotated(self):
148
134
        """Return merge with conflicts, showing origin of lines.
149
135
 
150
 
        Most useful for debugging merge.
 
136
        Most useful for debugging merge.        
151
137
        """
152
138
        for t in self.merge_regions():
153
139
            what = t[0]
234
220
 
235
221
        # section a[0:ia] has been disposed of, etc
236
222
        iz = ia = ib = 0
237
 
 
 
223
        
238
224
        for zmatch, zend, amatch, aend, bmatch, bend in self.find_sync_regions():
 
225
            #print 'match base [%d:%d]' % (zmatch, zend)
 
226
            
239
227
            matchlen = zend - zmatch
240
 
            # invariants:
241
 
            #   matchlen >= 0
242
 
            #   matchlen == (aend - amatch)
243
 
            #   matchlen == (bend - bmatch)
 
228
            assert matchlen >= 0
 
229
            assert matchlen == (aend - amatch)
 
230
            assert matchlen == (bend - bmatch)
 
231
            
244
232
            len_a = amatch - ia
245
233
            len_b = bmatch - ib
246
234
            len_base = zmatch - iz
247
 
            # invariants:
248
 
            # assert len_a >= 0
249
 
            # assert len_b >= 0
250
 
            # assert len_base >= 0
 
235
            assert len_a >= 0
 
236
            assert len_b >= 0
 
237
            assert len_base >= 0
251
238
 
252
239
            #print 'unmatched a=%d, b=%d' % (len_a, len_b)
253
240
 
286
273
            # that's OK, we can just skip it.
287
274
 
288
275
            if matchlen > 0:
289
 
                # invariants:
290
 
                # assert ia == amatch
291
 
                # assert ib == bmatch
292
 
                # assert iz == zmatch
293
 
 
 
276
                assert ia == amatch
 
277
                assert ib == bmatch
 
278
                assert iz == zmatch
 
279
                
294
280
                yield 'unchanged', zmatch, zend
295
281
                iz = zend
296
282
                ia = aend
299
285
    def _refine_cherrypick_conflict(self, zstart, zend, astart, aend, bstart, bend):
300
286
        """When cherrypicking b => a, ignore matches with b and base."""
301
287
        # Do not emit regions which match, only regions which do not match
302
 
        matches = patiencediff.PatienceSequenceMatcher(None,
 
288
        matches = bzrlib.patiencediff.PatienceSequenceMatcher(None,
303
289
            self.base[zstart:zend], self.b[bstart:bend]).get_matching_blocks()
304
 
        last_base_idx = 0
305
 
        last_b_idx = 0
306
 
        last_b_idx = 0
 
290
        zzcur = 0
 
291
        bbcur = 0
307
292
        yielded_a = False
308
 
        for base_idx, b_idx, match_len in matches:
309
 
            conflict_z_len = base_idx - last_base_idx
310
 
            conflict_b_len = b_idx - last_b_idx
 
293
        for region_iz, region_ib, region_len in matches:
 
294
            conflict_z_len = region_iz - zzcur
 
295
            conflict_b_len = region_ib - bbcur
311
296
            if conflict_b_len == 0: # There are no lines in b which conflict,
312
297
                                    # so skip it
313
298
                pass
314
299
            else:
315
300
                if yielded_a:
316
 
                    yield ('conflict',
317
 
                           zstart + last_base_idx, zstart + base_idx,
318
 
                           aend, aend, bstart + last_b_idx, bstart + b_idx)
 
301
                    yield ('conflict', zstart + zzcur, zstart + region_iz,
 
302
                           aend, aend, bstart + bbcur, bstart + region_ib)
319
303
                else:
320
304
                    # The first conflict gets the a-range
321
305
                    yielded_a = True
322
 
                    yield ('conflict', zstart + last_base_idx, zstart +
323
 
                    base_idx,
324
 
                           astart, aend, bstart + last_b_idx, bstart + b_idx)
325
 
            last_base_idx = base_idx + match_len
326
 
            last_b_idx = b_idx + match_len
327
 
        if last_base_idx != zend - zstart or last_b_idx != bend - bstart:
 
306
                    yield ('conflict', zstart + zzcur, zstart + region_iz,
 
307
                           astart, aend, bstart + bbcur, bstart + region_ib)
 
308
            zzcur = region_iz + region_len
 
309
            bbcur = region_ib + region_len
 
310
        if zzcur != zend - zstart or bbcur != bend - bstart:
328
311
            if yielded_a:
329
 
                yield ('conflict', zstart + last_base_idx, zstart + base_idx,
330
 
                       aend, aend, bstart + last_b_idx, bstart + b_idx)
 
312
                yield ('conflict', zstart + zzcur, zstart + region_iz,
 
313
                       aend, aend, bstart + bbcur, bstart + region_ib)
331
314
            else:
332
315
                # The first conflict gets the a-range
333
316
                yielded_a = True
334
 
                yield ('conflict', zstart + last_base_idx, zstart + base_idx,
335
 
                       astart, aend, bstart + last_b_idx, bstart + b_idx)
 
317
                yield ('conflict', zstart + zzcur, zstart + region_iz,
 
318
                       astart, aend, bstart + bbcur, bstart + region_ib)
336
319
        if not yielded_a:
337
320
            yield ('conflict', zstart, zend, astart, aend, bstart, bend)
338
321
 
339
322
    def reprocess_merge_regions(self, merge_regions):
340
323
        """Where there are conflict regions, remove the agreed lines.
341
324
 
342
 
        Lines where both A and B have made the same changes are
 
325
        Lines where both A and B have made the same changes are 
343
326
        eliminated.
344
327
        """
345
328
        for region in merge_regions:
349
332
            type, iz, zmatch, ia, amatch, ib, bmatch = region
350
333
            a_region = self.a[ia:amatch]
351
334
            b_region = self.b[ib:bmatch]
352
 
            matches = patiencediff.PatienceSequenceMatcher(
 
335
            matches = bzrlib.patiencediff.PatienceSequenceMatcher(
353
336
                    None, a_region, b_region).get_matching_blocks()
354
337
            next_a = ia
355
338
            next_b = ib
380
363
        """
381
364
 
382
365
        ia = ib = 0
383
 
        amatches = patiencediff.PatienceSequenceMatcher(
 
366
        amatches = bzrlib.patiencediff.PatienceSequenceMatcher(
384
367
                None, self.base, self.a).get_matching_blocks()
385
 
        bmatches = patiencediff.PatienceSequenceMatcher(
 
368
        bmatches = bzrlib.patiencediff.PatienceSequenceMatcher(
386
369
                None, self.base, self.b).get_matching_blocks()
387
370
        len_a = len(amatches)
388
371
        len_b = len(bmatches)
403
386
 
404
387
                # found a match of base[i[0], i[1]]; this may be less than
405
388
                # the region that matches in either one
406
 
                # assert intlen <= alen
407
 
                # assert intlen <= blen
408
 
                # assert abase <= intbase
409
 
                # assert bbase <= intbase
 
389
                assert intlen <= alen
 
390
                assert intlen <= blen
 
391
                assert abase <= intbase
 
392
                assert bbase <= intbase
410
393
 
411
394
                asub = amatch + (intbase - abase)
412
395
                bsub = bmatch + (intbase - bbase)
413
396
                aend = asub + intlen
414
397
                bend = bsub + intlen
415
398
 
416
 
                # assert self.base[intbase:intend] == self.a[asub:aend], \
417
 
                #       (self.base[intbase:intend], self.a[asub:aend])
418
 
                # assert self.base[intbase:intend] == self.b[bsub:bend]
 
399
                # XXX: How much overhead is involved in slicing all of these
 
400
                #      and doing an extra comparison
 
401
                assert self.base[intbase:intend] == self.a[asub:aend], \
 
402
                       (self.base[intbase:intend], self.a[asub:aend])
 
403
 
 
404
                assert self.base[intbase:intend] == self.b[bsub:bend]
419
405
 
420
406
                sl.append((intbase, intend,
421
407
                           asub, aend,
422
408
                           bsub, bend))
 
409
 
423
410
            # advance whichever one ends first in the base text
424
411
            if (abase + alen) < (bbase + blen):
425
412
                ia += 1
426
413
            else:
427
414
                ib += 1
428
 
 
 
415
            
429
416
        intbase = len(self.base)
430
417
        abase = len(self.a)
431
418
        bbase = len(self.b)
435
422
 
436
423
    def find_unconflicted(self):
437
424
        """Return a list of ranges in base that are not conflicted."""
438
 
        am = patiencediff.PatienceSequenceMatcher(
 
425
        am = bzrlib.patiencediff.PatienceSequenceMatcher(
439
426
                None, self.base, self.a).get_matching_blocks()
440
 
        bm = patiencediff.PatienceSequenceMatcher(
 
427
        bm = bzrlib.patiencediff.PatienceSequenceMatcher(
441
428
                None, self.base, self.b).get_matching_blocks()
442
429
 
443
430
        unc = []
457
444
                del am[0]
458
445
            else:
459
446
                del bm[0]
460
 
 
 
447
                
461
448
        return unc
462
449
 
463
450