/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/weave.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/weave.py
16
16
 
17
17
# Author: Martin Pool <mbp@canonical.com>
18
18
 
 
19
 
19
20
"""Weave - storage of related text file versions"""
20
21
 
21
 
from __future__ import absolute_import
22
22
 
23
23
# XXX: If we do weaves this way, will a merge still behave the same
24
24
# way if it's done in a different order?  That's a pretty desirable
67
67
# FIXME: the conflict markers should be *7* characters
68
68
 
69
69
from copy import copy
 
70
from cStringIO import StringIO
70
71
import os
71
72
 
72
 
from ..lazy_import import lazy_import
 
73
from bzrlib.lazy_import import lazy_import
73
74
lazy_import(globals(), """
74
 
from breezy import tsort
 
75
from bzrlib import tsort
75
76
""")
76
 
from .. import (
 
77
from bzrlib import (
77
78
    errors,
78
79
    osutils,
79
80
    )
80
 
from ..errors import (
81
 
    RevisionAlreadyPresent,
82
 
    RevisionNotPresent,
83
 
    UnavailableRepresentation,
84
 
    )
85
 
from ..osutils import dirname, sha, sha_strings, split_lines
86
 
from .. import patiencediff
87
 
from ..revision import NULL_REVISION
88
 
from ..sixish import (
89
 
    BytesIO,
90
 
    )
91
 
from ..trace import mutter
92
 
from .versionedfile import (
 
81
from bzrlib.errors import (WeaveError, WeaveFormatError, WeaveParentMismatch,
 
82
        RevisionAlreadyPresent,
 
83
        RevisionNotPresent,
 
84
        UnavailableRepresentation,
 
85
        )
 
86
from bzrlib.osutils import dirname, sha, sha_strings, split_lines
 
87
import bzrlib.patiencediff
 
88
from bzrlib.revision import NULL_REVISION
 
89
from bzrlib.symbol_versioning import *
 
90
from bzrlib.trace import mutter
 
91
from bzrlib.versionedfile import (
93
92
    AbsentContentFactory,
94
93
    adapter_registry,
95
94
    ContentFactory,
96
95
    sort_groupcompress,
97
96
    VersionedFile,
98
97
    )
99
 
from .weavefile import _read_weave_v5, write_weave_v5
100
 
 
101
 
 
102
 
class WeaveError(errors.BzrError):
103
 
 
104
 
    _fmt = "Error in processing weave: %(msg)s"
105
 
 
106
 
    def __init__(self, msg=None):
107
 
        errors.BzrError.__init__(self)
108
 
        self.msg = msg
109
 
 
110
 
 
111
 
class WeaveRevisionAlreadyPresent(WeaveError):
112
 
 
113
 
    _fmt = "Revision {%(revision_id)s} already present in %(weave)s"
114
 
 
115
 
    def __init__(self, revision_id, weave):
116
 
 
117
 
        WeaveError.__init__(self)
118
 
        self.revision_id = revision_id
119
 
        self.weave = weave
120
 
 
121
 
 
122
 
class WeaveRevisionNotPresent(WeaveError):
123
 
 
124
 
    _fmt = "Revision {%(revision_id)s} not present in %(weave)s"
125
 
 
126
 
    def __init__(self, revision_id, weave):
127
 
        WeaveError.__init__(self)
128
 
        self.revision_id = revision_id
129
 
        self.weave = weave
130
 
 
131
 
 
132
 
class WeaveFormatError(WeaveError):
133
 
 
134
 
    _fmt = "Weave invariant violated: %(what)s"
135
 
 
136
 
    def __init__(self, what):
137
 
        WeaveError.__init__(self)
138
 
        self.what = what
139
 
 
140
 
 
141
 
class WeaveParentMismatch(WeaveError):
142
 
 
143
 
    _fmt = "Parents are mismatched between two revisions. %(msg)s"
144
 
 
145
 
 
146
 
class WeaveInvalidChecksum(WeaveError):
147
 
 
148
 
    _fmt = "Text did not match its checksum: %(msg)s"
149
 
 
150
 
 
151
 
class WeaveTextDiffers(WeaveError):
152
 
 
153
 
    _fmt = ("Weaves differ on text content. Revision:"
154
 
            " {%(revision_id)s}, %(weave_a)s, %(weave_b)s")
155
 
 
156
 
    def __init__(self, revision_id, weave_a, weave_b):
157
 
        WeaveError.__init__(self)
158
 
        self.revision_id = revision_id
159
 
        self.weave_a = weave_a
160
 
        self.weave_b = weave_b
 
98
from bzrlib.weavefile import _read_weave_v5, write_weave_v5
161
99
 
162
100
 
163
101
class WeaveContentFactory(ContentFactory):
293
231
        self._name_map = {}
294
232
        self._weave_name = weave_name
295
233
        if matcher is None:
296
 
            self._matcher = patiencediff.PatienceSequenceMatcher
 
234
            self._matcher = bzrlib.patiencediff.PatienceSequenceMatcher
297
235
        else:
298
236
            self._matcher = matcher
299
237
        if get_scope is None:
300
 
            def get_scope():
301
 
                return None
 
238
            get_scope = lambda:None
302
239
        self._get_scope = get_scope
303
240
        self._scope = get_scope()
304
241
        self._access_mode = access_mode
331
268
        if not isinstance(other, Weave):
332
269
            return False
333
270
        return self._parents == other._parents \
334
 
            and self._weave == other._weave \
335
 
            and self._sha1s == other._sha1s
 
271
               and self._weave == other._weave \
 
272
               and self._sha1s == other._sha1s
336
273
 
337
274
    def __ne__(self, other):
338
275
        return not self.__eq__(other)
422
359
                raise RevisionNotPresent([record.key[0]], self)
423
360
            # adapt to non-tuple interface
424
361
            parents = [parent[0] for parent in record.parents]
425
 
            if (record.storage_kind == 'fulltext' or
426
 
                    record.storage_kind == 'chunked'):
427
 
                self.add_lines(
428
 
                    record.key[0], parents,
 
362
            if (record.storage_kind == 'fulltext'
 
363
                or record.storage_kind == 'chunked'):
 
364
                self.add_lines(record.key[0], parents,
429
365
                    osutils.chunks_to_lines(record.get_bytes_as('chunked')))
430
366
            else:
431
367
                adapter_key = record.storage_kind, 'fulltext'
448
384
        """
449
385
        idx = self._lookup(name)
450
386
        if sorted(self._parents[idx]) != sorted(parents) \
451
 
                or sha1 != self._sha1s[idx]:
 
387
            or sha1 != self._sha1s[idx]:
452
388
            raise RevisionAlreadyPresent(name, self._weave_name)
453
389
        return idx
454
390
 
455
391
    def _add_lines(self, version_id, parents, lines, parent_texts,
456
 
                   left_matching_blocks, nostore_sha, random_id,
457
 
                   check_content):
 
392
       left_matching_blocks, nostore_sha, random_id, check_content):
458
393
        """See VersionedFile.add_lines."""
459
 
        idx = self._add(version_id, lines, list(map(self._lookup, parents)),
460
 
                        nostore_sha=nostore_sha)
 
394
        idx = self._add(version_id, lines, map(self._lookup, parents),
 
395
            nostore_sha=nostore_sha)
461
396
        return sha_strings(lines), sum(map(len, lines)), idx
462
397
 
463
398
    def _add(self, version_id, lines, parents, sha1=None, nostore_sha=None):
485
420
        if sha1 == nostore_sha:
486
421
            raise errors.ExistingContent
487
422
        if version_id is None:
488
 
            version_id = b"sha1:" + sha1
 
423
            version_id = "sha1:" + sha1
489
424
        if version_id in self._name_map:
490
425
            return self._check_repeated_add(version_id, parents, lines, sha1)
491
426
 
492
427
        self._check_versions(parents)
 
428
        ## self._check_lines(lines)
493
429
        new_version = len(self._parents)
494
430
 
495
 
        # if we abort after here the (in-memory) weave will be corrupt because
496
 
        # only some fields are updated
 
431
        # if we abort after here the (in-memory) weave will be corrupt because only
 
432
        # some fields are updated
497
433
        # XXX: FIXME implement a succeed-or-fail of the rest of this routine.
498
434
        #      - Robert Collins 20060226
499
435
        self._parents.append(parents[:])
501
437
        self._names.append(version_id)
502
438
        self._name_map[version_id] = new_version
503
439
 
 
440
 
504
441
        if not parents:
505
442
            # special case; adding with no parents revision; can do
506
443
            # this more quickly by just appending unconditionally.
507
444
            # even more specially, if we're adding an empty text we
508
445
            # need do nothing at all.
509
446
            if lines:
510
 
                self._weave.append((b'{', new_version))
 
447
                self._weave.append(('{', new_version))
511
448
                self._weave.extend(lines)
512
 
                self._weave.append((b'}', None))
 
449
                self._weave.append(('}', None))
513
450
            return new_version
514
451
 
515
452
        if len(parents) == 1:
518
455
                # special case: same as the single parent
519
456
                return new_version
520
457
 
 
458
 
521
459
        ancestors = self._inclusions(parents)
522
460
 
523
461
        l = self._weave
541
479
        # matches the end of the file?  the current code says it's the
542
480
        # last line of the weave?
543
481
 
544
 
        # print 'basis_lines:', basis_lines
545
 
        # print 'new_lines:  ', lines
 
482
        #print 'basis_lines:', basis_lines
 
483
        #print 'new_lines:  ', lines
546
484
 
547
485
        s = self._matcher(None, basis_lines, lines)
548
486
 
549
487
        # offset gives the number of lines that have been inserted
550
 
        # into the weave up to the current point; if the original edit
551
 
        # instruction says to change line A then we actually change (A+offset)
 
488
        # into the weave up to the current point; if the original edit instruction
 
489
        # says to change line A then we actually change (A+offset)
552
490
        offset = 0
553
491
 
554
492
        for tag, i1, i2, j1, j2 in s.get_opcodes():
555
 
            # i1,i2 are given in offsets within basis_lines; we need to map
556
 
            # them back to offsets within the entire weave print 'raw match',
557
 
            # tag, i1, i2, j1, j2
 
493
            # i1,i2 are given in offsets within basis_lines; we need to map them
 
494
            # back to offsets within the entire weave
 
495
            #print 'raw match', tag, i1, i2, j1, j2
558
496
            if tag == 'equal':
559
497
                continue
560
498
            i1 = basis_lineno[i1]
562
500
            # the deletion and insertion are handled separately.
563
501
            # first delete the region.
564
502
            if i1 != i2:
565
 
                self._weave.insert(i1 + offset, (b'[', new_version))
566
 
                self._weave.insert(i2 + offset + 1, (b']', new_version))
 
503
                self._weave.insert(i1+offset, ('[', new_version))
 
504
                self._weave.insert(i2+offset+1, (']', new_version))
567
505
                offset += 2
568
506
 
569
507
            if j1 != j2:
571
509
                # i2; we want to insert after this region to make sure
572
510
                # we don't destroy ourselves
573
511
                i = i2 + offset
574
 
                self._weave[i:i] = ([(b'{', new_version)] +
575
 
                                    lines[j1:j2] +
576
 
                                    [(b'}', None)])
 
512
                self._weave[i:i] = ([('{', new_version)]
 
513
                                    + lines[j1:j2]
 
514
                                    + [('}', None)])
577
515
                offset += 2 + (j2 - j1)
578
516
        return new_version
579
517
 
582
520
        if not len(versions):
583
521
            return []
584
522
        i = set(versions)
585
 
        for v in range(max(versions), 0, -1):
 
523
        for v in xrange(max(versions), 0, -1):
586
524
            if v in i:
587
525
                # include all its parents
588
526
                i.update(self._parents[v])
589
527
        return i
 
528
        ## except IndexError:
 
529
        ##     raise ValueError("version %d not present in weave" % v)
590
530
 
591
531
    def get_ancestry(self, version_ids, topo_sorted=True):
592
532
        """See VersionedFile.get_ancestry."""
593
 
        if isinstance(version_ids, bytes):
 
533
        if isinstance(version_ids, basestring):
594
534
            version_ids = [version_ids]
595
535
        i = self._inclusions([self._lookup(v) for v in version_ids])
596
536
        return [self._idx_to_name(v) for v in i]
597
537
 
 
538
    def _check_lines(self, text):
 
539
        if not isinstance(text, list):
 
540
            raise ValueError("text should be a list, not %s" % type(text))
 
541
 
 
542
        for l in text:
 
543
            if not isinstance(l, basestring):
 
544
                raise ValueError("text line should be a string or unicode, not %s"
 
545
                                 % type(l))
 
546
 
 
547
 
 
548
 
598
549
    def _check_versions(self, indexes):
599
550
        """Check everything in the sequence of indexes is valid"""
600
551
        for i in indexes:
617
568
        The index indicates when the line originated in the weave."""
618
569
        incls = [self._lookup(version_id)]
619
570
        return [(self._idx_to_name(origin), text) for origin, lineno, text in
620
 
                self._extract(incls)]
 
571
            self._extract(incls)]
621
572
 
622
573
    def iter_lines_added_or_present_in_versions(self, version_ids=None,
623
574
                                                pb=None):
625
576
        if version_ids is None:
626
577
            version_ids = self.versions()
627
578
        version_ids = set(version_ids)
628
 
        for lineno, inserted, deletes, line in self._walk_internal(
629
 
                version_ids):
630
 
            if inserted not in version_ids:
631
 
                continue
632
 
            if not line.endswith(b'\n'):
633
 
                yield line + b'\n', inserted
 
579
        for lineno, inserted, deletes, line in self._walk_internal(version_ids):
 
580
            if inserted not in version_ids: continue
 
581
            if line[-1] != '\n':
 
582
                yield line + '\n', inserted
634
583
            else:
635
584
                yield line, inserted
636
585
 
645
594
        for l in self._weave:
646
595
            if l.__class__ == tuple:
647
596
                c, v = l
648
 
                if c == b'{':
 
597
                isactive = None
 
598
                if c == '{':
649
599
                    istack.append(self._names[v])
650
 
                elif c == b'}':
 
600
                elif c == '}':
651
601
                    istack.pop()
652
 
                elif c == b'[':
 
602
                elif c == '[':
653
603
                    dset.add(self._names[v])
654
 
                elif c == b']':
 
604
                elif c == ']':
655
605
                    dset.remove(self._names[v])
656
606
                else:
657
607
                    raise WeaveFormatError('unexpected instruction %r' % v)
661
611
 
662
612
        if istack:
663
613
            raise WeaveFormatError("unclosed insertion blocks "
664
 
                                   "at end of weave: %s" % istack)
 
614
                    "at end of weave: %s" % istack)
665
615
        if dset:
666
 
            raise WeaveFormatError(
667
 
                "unclosed deletion blocks at end of weave: %s" % dset)
 
616
            raise WeaveFormatError("unclosed deletion blocks at end of weave: %s"
 
617
                                   % dset)
668
618
 
669
619
    def plan_merge(self, ver_a, ver_b):
670
620
        """Return pseudo-annotation indicating how the two versions merge.
678
628
        inc_b = set(self.get_ancestry([ver_b]))
679
629
        inc_c = inc_a & inc_b
680
630
 
681
 
        for lineno, insert, deleteset, line in self._walk_internal(
682
 
                [ver_a, ver_b]):
 
631
        for lineno, insert, deleteset, line in self._walk_internal([ver_a, ver_b]):
683
632
            if deleteset & inc_c:
684
633
                # killed in parent; can't be in either a or b
685
634
                # not relevant to our work
736
685
 
737
686
        result = []
738
687
 
 
688
        WFE = WeaveFormatError
 
689
 
739
690
        # wow.
740
 
        #  449       0   4474.6820   2356.5590   breezy.weave:556(_extract)
 
691
        #  449       0   4474.6820   2356.5590   bzrlib.weave:556(_extract)
741
692
        #  +285282   0   1676.8040   1676.8040   +<isinstance>
742
693
        # 1.6 seconds in 'isinstance'.
743
694
        # changing the first isinstance:
744
 
        #  449       0   2814.2660   1577.1760   breezy.weave:556(_extract)
 
695
        #  449       0   2814.2660   1577.1760   bzrlib.weave:556(_extract)
745
696
        #  +140414   0    762.8050    762.8050   +<isinstance>
746
697
        # note that the inline time actually dropped (less function calls)
747
698
        # and total processing time was halved.
748
699
        # we're still spending ~1/4 of the method in isinstance though.
749
700
        # so lets hard code the acceptable string classes we expect:
750
 
        #  449       0   1202.9420    786.2930   breezy.weave:556(_extract)
 
701
        #  449       0   1202.9420    786.2930   bzrlib.weave:556(_extract)
751
702
        # +71352     0    377.5560    377.5560   +<method 'append' of 'list'
752
703
        #                                          objects>
753
704
        # yay, down to ~1/4 the initial extract time, and our inline time
754
705
        # has shrunk again, with isinstance no longer dominating.
755
706
        # tweaking the stack inclusion test to use a set gives:
756
 
        #  449       0   1122.8030    713.0080   breezy.weave:556(_extract)
 
707
        #  449       0   1122.8030    713.0080   bzrlib.weave:556(_extract)
757
708
        # +71352     0    354.9980    354.9980   +<method 'append' of 'list'
758
709
        #                                          objects>
759
710
        # - a 5% win, or possibly just noise. However with large istacks that
760
 
        # 'in' test could dominate, so I'm leaving this change in place - when
761
 
        # its fast enough to consider profiling big datasets we can review.
 
711
        # 'in' test could dominate, so I'm leaving this change in place -
 
712
        # when its fast enough to consider profiling big datasets we can review.
 
713
 
 
714
 
 
715
 
762
716
 
763
717
        for l in self._weave:
764
718
            if l.__class__ == tuple:
765
719
                c, v = l
766
720
                isactive = None
767
 
                if c == b'{':
 
721
                if c == '{':
768
722
                    istack.append(v)
769
723
                    iset.add(v)
770
 
                elif c == b'}':
 
724
                elif c == '}':
771
725
                    iset.remove(istack.pop())
772
 
                elif c == b'[':
 
726
                elif c == '[':
773
727
                    if v in included:
774
728
                        dset.add(v)
775
 
                elif c == b']':
 
729
                elif c == ']':
776
730
                    if v in included:
777
731
                        dset.remove(v)
778
732
                else:
779
733
                    raise AssertionError()
780
734
            else:
781
735
                if isactive is None:
782
 
                    isactive = (not dset) and istack and (
783
 
                        istack[-1] in included)
 
736
                    isactive = (not dset) and istack and (istack[-1] in included)
784
737
                if isactive:
785
738
                    result.append((istack[-1], lineno, l))
786
739
            lineno += 1
787
740
        if istack:
788
741
            raise WeaveFormatError("unclosed insertion blocks "
789
 
                                   "at end of weave: %s" % istack)
 
742
                    "at end of weave: %s" % istack)
790
743
        if dset:
791
 
            raise WeaveFormatError(
792
 
                "unclosed deletion blocks at end of weave: %s" % dset)
 
744
            raise WeaveFormatError("unclosed deletion blocks at end of weave: %s"
 
745
                                   % dset)
793
746
        return result
794
747
 
795
748
    def _maybe_lookup(self, name_or_index):
797
750
 
798
751
        NOT FOR PUBLIC USE.
799
752
        """
800
 
        # GZ 2017-04-01: This used to check for long as well, but I don't think
801
 
        # there are python implementations with sys.maxsize > sys.maxint
802
 
        if isinstance(name_or_index, int):
 
753
        if isinstance(name_or_index, (int, long)):
803
754
            return name_or_index
804
755
        else:
805
756
            return self._lookup(name_or_index)
807
758
    def get_lines(self, version_id):
808
759
        """See VersionedFile.get_lines()."""
809
760
        int_index = self._maybe_lookup(version_id)
810
 
        result = [line for (origin, lineno, line)
811
 
                  in self._extract([int_index])]
 
761
        result = [line for (origin, lineno, line) in self._extract([int_index])]
812
762
        expected_sha1 = self._sha1s[int_index]
813
763
        measured_sha1 = sha_strings(result)
814
764
        if measured_sha1 != expected_sha1:
815
 
            raise WeaveInvalidChecksum(
816
 
                'file %s, revision %s, expected: %s, measured %s'
817
 
                % (self._weave_name, version_id,
818
 
                   expected_sha1, measured_sha1))
 
765
            raise errors.WeaveInvalidChecksum(
 
766
                    'file %s, revision %s, expected: %s, measured %s'
 
767
                    % (self._weave_name, version_id,
 
768
                       expected_sha1, measured_sha1))
819
769
        return result
820
770
 
821
771
    def get_sha1s(self, version_ids):
827
777
 
828
778
    def num_versions(self):
829
779
        """How many versions are in this weave?"""
830
 
        return len(self._parents)
 
780
        l = len(self._parents)
 
781
        return l
831
782
 
832
783
    __len__ = num_versions
833
784
 
840
791
            if inclusions:
841
792
                inclusions.sort()
842
793
                if inclusions[-1] >= version:
843
 
                    raise WeaveFormatError(
844
 
                        "invalid included version %d for index %d"
845
 
                        % (inclusions[-1], version))
 
794
                    raise WeaveFormatError("invalid included version %d for index %d"
 
795
                                           % (inclusions[-1], version))
846
796
 
847
797
        # try extracting all versions; parallel extraction is used
848
798
        nv = self.num_versions()
855
805
            name = self._idx_to_name(i)
856
806
            sha1s[name] = sha()
857
807
            texts[name] = []
858
 
            new_inc = {name}
 
808
            new_inc = set([name])
859
809
            for p in self._parents[i]:
860
810
                new_inc.update(inclusions[self._idx_to_name(p)])
861
811
 
881
831
                # The active inclusion must be an ancestor,
882
832
                # and no ancestors must have deleted this line,
883
833
                # because we don't support resurrection.
884
 
                if ((insert in name_inclusions) and
885
 
                        not (deleteset & name_inclusions)):
 
834
                if (insert in name_inclusions) and not (deleteset & name_inclusions):
886
835
                    sha1s[name].update(line)
887
836
 
888
837
        for i in range(nv):
889
838
            version = self._idx_to_name(i)
890
 
            hd = sha1s[version].hexdigest().encode()
 
839
            hd = sha1s[version].hexdigest()
891
840
            expected = self._sha1s[i]
892
841
            if hd != expected:
893
 
                raise WeaveInvalidChecksum(
894
 
                    "mismatched sha1 for version %s: "
895
 
                    "got %s, expected %s"
896
 
                    % (version, hd, expected))
 
842
                raise errors.WeaveInvalidChecksum(
 
843
                        "mismatched sha1 for version %s: "
 
844
                        "got %s, expected %s"
 
845
                        % (version, hd, expected))
897
846
 
898
847
        # TODO: check insertions are properly nested, that there are
899
848
        # no lines outside of insertion blocks, that deletions are
926
875
        this_idx = self._name_map.get(name, -1)
927
876
        if this_idx != -1:
928
877
            if self._sha1s[this_idx] != other._sha1s[other_idx]:
929
 
                raise WeaveTextDiffers(name, self, other)
 
878
                raise errors.WeaveTextDiffers(name, self, other)
930
879
            self_parents = self._parents[this_idx]
931
880
            other_parents = other._parents[other_idx]
932
 
            n1 = {self._names[i] for i in self_parents}
933
 
            n2 = {other._names[i] for i in other_parents}
 
881
            n1 = set([self._names[i] for i in self_parents])
 
882
            n2 = set([other._names[i] for i in other_parents])
934
883
            if not self._compatible_parents(n1, n2):
935
 
                raise WeaveParentMismatch(
936
 
                    "inconsistent parents "
 
884
                raise WeaveParentMismatch("inconsistent parents "
937
885
                    "for version {%s}: %s vs %s" % (name, n1, n2))
938
886
            else:
939
887
                return True         # ok!
962
910
 
963
911
    WEAVE_SUFFIX = '.weave'
964
912
 
965
 
    def __init__(self, name, transport, filemode=None, create=False,
966
 
                 access_mode='w', get_scope=None):
 
913
    def __init__(self, name, transport, filemode=None, create=False, access_mode='w', get_scope=None):
967
914
        """Create a WeaveFile.
968
915
 
969
916
        :param create: If not True, only open an existing knit.
970
917
        """
971
918
        super(WeaveFile, self).__init__(name, access_mode, get_scope=get_scope,
972
 
                                        allow_reserved=False)
 
919
            allow_reserved=False)
973
920
        self._transport = transport
974
921
        self._filemode = filemode
975
922
        try:
976
 
            f = self._transport.get(name + WeaveFile.WEAVE_SUFFIX)
977
 
            _read_weave_v5(BytesIO(f.read()), self)
 
923
            _read_weave_v5(self._transport.get(name + WeaveFile.WEAVE_SUFFIX), self)
978
924
        except errors.NoSuchFile:
979
925
            if not create:
980
926
                raise
982
928
            self._save()
983
929
 
984
930
    def _add_lines(self, version_id, parents, lines, parent_texts,
985
 
                   left_matching_blocks, nostore_sha, random_id,
986
 
                   check_content):
 
931
        left_matching_blocks, nostore_sha, random_id, check_content):
987
932
        """Add a version and save the weave."""
988
933
        self.check_not_reserved_id(version_id)
989
 
        result = super(WeaveFile, self)._add_lines(
990
 
            version_id, parents, lines, parent_texts, left_matching_blocks,
991
 
            nostore_sha, random_id, check_content)
 
934
        result = super(WeaveFile, self)._add_lines(version_id, parents, lines,
 
935
            parent_texts, left_matching_blocks, nostore_sha, random_id,
 
936
            check_content)
992
937
        self._save()
993
938
        return result
994
939
 
995
940
    def copy_to(self, name, transport):
996
941
        """See VersionedFile.copy_to()."""
997
942
        # as we are all in memory always, just serialise to the new place.
998
 
        sio = BytesIO()
 
943
        sio = StringIO()
999
944
        write_weave_v5(self, sio)
1000
945
        sio.seek(0)
1001
946
        transport.put_file(name + WeaveFile.WEAVE_SUFFIX, sio, self._filemode)
1003
948
    def _save(self):
1004
949
        """Save the weave."""
1005
950
        self._check_write_ok()
1006
 
        sio = BytesIO()
 
951
        sio = StringIO()
1007
952
        write_weave_v5(self, sio)
1008
953
        sio.seek(0)
1009
954
        bytes = sio.getvalue()
1040
985
    :param msg: An optional message for the progress
1041
986
    """
1042
987
    wr = Weave()
 
988
    ia = ib = 0
 
989
    queue_a = range(wa.num_versions())
 
990
    queue_b = range(wb.num_versions())
1043
991
    # first determine combined parents of all versions
1044
992
    # map from version name -> all parent names
1045
993
    combined_parents = _reweave_parent_graphs(wa, wb)
1046
994
    mutter("combined parents: %r", combined_parents)
1047
 
    order = tsort.topo_sort(combined_parents.items())
 
995
    order = tsort.topo_sort(combined_parents.iteritems())
1048
996
    mutter("order to reweave: %r", order)
1049
997
 
1050
998
    if pb and not msg:
1062
1010
                    mutter('weaves: %s, %s', wa._weave_name, wb._weave_name)
1063
1011
                    import difflib
1064
1012
                    lines = list(difflib.unified_diff(lines, lines_b,
1065
 
                                                      wa._weave_name, wb._weave_name))
 
1013
                            wa._weave_name, wb._weave_name))
1066
1014
                    mutter('lines:\n%s', ''.join(lines))
1067
 
                    raise WeaveTextDiffers(name, wa, wb)
 
1015
                    raise errors.WeaveTextDiffers(name, wa, wb)
1068
1016
        else:
1069
1017
            lines = wb.get_lines(name)
1070
1018
        wr._add(name, lines, [wr._lookup(i) for i in combined_parents[name]])