~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tsort.py

  • Committer: Jelmer Vernooij
  • Date: 2010-12-20 11:57:14 UTC
  • mto: This revision was merged to the branch mainline in revision 5577.
  • Revision ID: jelmer@samba.org-20101220115714-2ru3hfappjweeg7q
Don't use no-plugins.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2008 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., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
17
 
18
18
"""Topological sorting routines."""
19
19
 
20
20
 
21
 
from bzrlib import errors
 
21
from bzrlib import (
 
22
    errors,
 
23
    graph as _mod_graph,
 
24
    revision as _mod_revision,
 
25
    )
22
26
 
23
27
 
24
28
__all__ = ["topo_sort", "TopoSorter", "merge_sort", "MergeSorter"]
29
33
 
30
34
    graph -- sequence of pairs of node->parents_list.
31
35
 
32
 
    The result is a list of node names, such that all parents come before
33
 
    their children.
 
36
    The result is a list of node names, such that all parents come before their
 
37
    children.
34
38
 
35
39
    node identifiers can be any hashable object, and are typically strings.
 
40
 
 
41
    This function has the same purpose as the TopoSorter class, but uses a
 
42
    different algorithm to sort the graph. That means that while both return a
 
43
    list with parents before their child nodes, the exact ordering can be
 
44
    different.
 
45
 
 
46
    topo_sort is faster when the whole list is needed, while when iterating
 
47
    over a part of the list, TopoSorter.iter_topo_order should be used.
36
48
    """
37
 
    return TopoSorter(graph).sorted()
 
49
    kg = _mod_graph.KnownGraph(dict(graph))
 
50
    return kg.topo_sort()
38
51
 
39
52
 
40
53
class TopoSorter(object):
41
54
 
42
55
    def __init__(self, graph):
43
56
        """Topological sorting of a graph.
44
 
    
 
57
 
45
58
        :param graph: sequence of pairs of node_name->parent_names_list.
46
59
                      i.e. [('C', ['B']), ('B', ['A']), ('A', [])]
47
60
                      For this input the output from the sort or
48
61
                      iter_topo_order routines will be:
49
62
                      'A', 'B', 'C'
50
 
        
 
63
 
51
64
        node identifiers can be any hashable object, and are typically strings.
52
65
 
53
66
        If you have a graph like [('a', ['b']), ('a', ['c'])] this will only use
56
69
        The graph is sorted lazily: until you iterate or sort the input is
57
70
        not processed other than to create an internal representation.
58
71
 
59
 
        iteration or sorting may raise GraphCycleError if a cycle is present 
 
72
        iteration or sorting may raise GraphCycleError if a cycle is present
60
73
        in the graph.
61
74
        """
62
 
        # a dict of the graph.
 
75
        # store a dict of the graph.
63
76
        self._graph = dict(graph)
64
 
        self._visitable = set(self._graph)
65
 
        ### if debugging:
66
 
        # self._original_graph = dict(graph)
67
 
        
68
 
        # this is a stack storing the depth first search into the graph.
69
 
        self._node_name_stack = []
70
 
        # at each level of 'recursion' we have to check each parent. This
71
 
        # stack stores the parents we have not yet checked for the node at the 
72
 
        # matching depth in _node_name_stack
73
 
        self._pending_parents_stack = []
74
 
        # this is a set of the completed nodes for fast checking whether a
75
 
        # parent in a node we are processing on the stack has already been
76
 
        # emitted and thus can be skipped.
77
 
        self._completed_node_names = set()
78
77
 
79
78
    def sorted(self):
80
79
        """Sort the graph and return as a list.
81
 
        
 
80
 
82
81
        After calling this the sorter is empty and you must create a new one.
83
82
        """
84
83
        return list(self.iter_topo_order())
95
94
 
96
95
    def iter_topo_order(self):
97
96
        """Yield the nodes of the graph in a topological order.
98
 
        
 
97
 
99
98
        After finishing iteration the sorter is empty and you cannot continue
100
99
        iteration.
101
100
        """
102
 
        while self._graph:
 
101
        graph = self._graph
 
102
        visitable = set(graph)
 
103
 
 
104
        # this is a stack storing the depth first search into the graph.
 
105
        pending_node_stack = []
 
106
        # at each level of 'recursion' we have to check each parent. This
 
107
        # stack stores the parents we have not yet checked for the node at the
 
108
        # matching depth in pending_node_stack
 
109
        pending_parents_stack = []
 
110
 
 
111
        # this is a set of the completed nodes for fast checking whether a
 
112
        # parent in a node we are processing on the stack has already been
 
113
        # emitted and thus can be skipped.
 
114
        completed_node_names = set()
 
115
 
 
116
        while graph:
103
117
            # now pick a random node in the source graph, and transfer it to the
104
 
            # top of the depth first search stack.
105
 
            node_name, parents = self._graph.popitem()
106
 
            self._push_node(node_name, parents)
107
 
            while self._node_name_stack:
108
 
                # loop until this call completes.
109
 
                parents_to_visit = self._pending_parents_stack[-1]
110
 
                # if all parents are done, the revision is done
 
118
            # top of the depth first search stack of pending nodes.
 
119
            node_name, parents = graph.popitem()
 
120
            pending_node_stack.append(node_name)
 
121
            pending_parents_stack.append(list(parents))
 
122
 
 
123
            # loop until pending_node_stack is empty
 
124
            while pending_node_stack:
 
125
                parents_to_visit = pending_parents_stack[-1]
 
126
                # if there are no parents left, the revision is done
111
127
                if not parents_to_visit:
112
128
                    # append the revision to the topo sorted list
113
 
                    # all the nodes parents have been added to the output, now
114
 
                    # we can add it to the output.
115
 
                    yield self._pop_node()
 
129
                    # all the nodes parents have been added to the output,
 
130
                    # now we can add it to the output.
 
131
                    popped_node = pending_node_stack.pop()
 
132
                    pending_parents_stack.pop()
 
133
                    completed_node_names.add(popped_node)
 
134
                    yield popped_node
116
135
                else:
117
 
                    while self._pending_parents_stack[-1]:
118
 
                        # recurse depth first into a single parent 
119
 
                        next_node_name = self._pending_parents_stack[-1].pop()
120
 
                        if next_node_name in self._completed_node_names:
121
 
                            # this parent was completed by a child on the
122
 
                            # call stack. skip it.
123
 
                            continue
124
 
                        if next_node_name not in self._visitable:
125
 
                            continue
126
 
                        # otherwise transfer it from the source graph into the
127
 
                        # top of the current depth first search stack.
128
 
                        try:
129
 
                            parents = self._graph.pop(next_node_name)
130
 
                        except KeyError:
131
 
                            # if the next node is not in the source graph it has
132
 
                            # already been popped from it and placed into the
133
 
                            # current search stack (but not completed or we would
134
 
                            # have hit the continue 4 lines up.
135
 
                            # this indicates a cycle.
136
 
                            raise errors.GraphCycleError(self._node_name_stack)
137
 
                        self._push_node(next_node_name, parents)
138
 
                        # and do not continue processing parents until this 'call' 
139
 
                        # has recursed.
140
 
                        break
141
 
 
142
 
    def _push_node(self, node_name, parents):
143
 
        """Add node_name to the pending node stack.
144
 
        
145
 
        Names in this stack will get emitted into the output as they are popped
146
 
        off the stack.
147
 
        """
148
 
        self._node_name_stack.append(node_name)
149
 
        self._pending_parents_stack.append(list(parents))
150
 
 
151
 
    def _pop_node(self):
152
 
        """Pop the top node off the stack 
153
 
 
154
 
        The node is appended to the sorted output.
155
 
        """
156
 
        # we are returning from the flattened call frame:
157
 
        # pop off the local variables
158
 
        node_name = self._node_name_stack.pop()
159
 
        self._pending_parents_stack.pop()
160
 
 
161
 
        self._completed_node_names.add(node_name)
162
 
        return node_name
 
136
                    # recurse depth first into a single parent
 
137
                    next_node_name = parents_to_visit.pop()
 
138
 
 
139
                    if next_node_name in completed_node_names:
 
140
                        # parent was already completed by a child, skip it.
 
141
                        continue
 
142
                    if next_node_name not in visitable:
 
143
                        # parent is not a node in the original graph, skip it.
 
144
                        continue
 
145
 
 
146
                    # transfer it along with its parents from the source graph
 
147
                    # into the top of the current depth first search stack.
 
148
                    try:
 
149
                        parents = graph.pop(next_node_name)
 
150
                    except KeyError:
 
151
                        # if the next node is not in the source graph it has
 
152
                        # already been popped from it and placed into the
 
153
                        # current search stack (but not completed or we would
 
154
                        # have hit the continue 6 lines up).  this indicates a
 
155
                        # cycle.
 
156
                        raise errors.GraphCycleError(pending_node_stack)
 
157
                    pending_node_stack.append(next_node_name)
 
158
                    pending_parents_stack.append(list(parents))
163
159
 
164
160
 
165
161
def merge_sort(graph, branch_tip, mainline_revisions=None, generate_revno=False):
166
162
    """Topological sort a graph which groups merges.
167
163
 
168
164
    :param graph: sequence of pairs of node->parents_list.
169
 
    :param branch_tip: the tip of the branch to graph. Revisions not 
 
165
    :param branch_tip: the tip of the branch to graph. Revisions not
170
166
                       reachable from branch_tip are not included in the
171
167
                       output.
172
168
    :param mainline_revisions: If not None this forces a mainline to be
192
188
    __slots__ = ['_node_name_stack',
193
189
                 '_node_merge_depth_stack',
194
190
                 '_pending_parents_stack',
195
 
                 '_assigned_sequence_stack',
 
191
                 '_first_child_stack',
196
192
                 '_left_subtree_pushed_stack',
197
193
                 '_generate_revno',
198
194
                 '_graph',
200
196
                 '_stop_revision',
201
197
                 '_original_graph',
202
198
                 '_revnos',
203
 
                 '_root_sequence',
 
199
                 '_revno_to_branch_count',
204
200
                 '_completed_node_names',
205
201
                 '_scheduled_nodes',
206
202
                ]
208
204
    def __init__(self, graph, branch_tip, mainline_revisions=None,
209
205
        generate_revno=False):
210
206
        """Merge-aware topological sorting of a graph.
211
 
    
 
207
 
212
208
        :param graph: sequence of pairs of node_name->parent_names_list.
213
209
                      i.e. [('C', ['B']), ('B', ['A']), ('A', [])]
214
210
                      For this input the output from the sort or
215
211
                      iter_topo_order routines will be:
216
212
                      'A', 'B', 'C'
217
 
        :param branch_tip: the tip of the branch to graph. Revisions not 
 
213
        :param branch_tip: the tip of the branch to graph. Revisions not
218
214
                       reachable from branch_tip are not included in the
219
215
                       output.
220
216
        :param mainline_revisions: If not None this forces a mainline to be
232
228
        The result is a list sorted so that all parents come before
233
229
        their children. Each element of the list is a tuple containing:
234
230
        (sequence_number, node_name, merge_depth, end_of_merge)
235
 
         * sequence_number: The sequence of this row in the output. Useful for 
 
231
         * sequence_number: The sequence of this row in the output. Useful for
236
232
           GUIs.
237
233
         * node_name: The node name: opaque text to the merge routine.
238
234
         * merge_depth: How many levels of merging deep this node has been
239
235
           found.
240
236
         * revno_sequence: When requested this field provides a sequence of
241
237
             revision numbers for all revisions. The format is:
242
 
             REVNO[[.BRANCHREVNO.REVNO] ...]. BRANCHREVNO is the number of the
 
238
             (REVNO, BRANCHNUM, BRANCHREVNO). BRANCHNUM is the number of the
243
239
             branch that the revno is on. From left to right the REVNO numbers
244
240
             are the sequence numbers within that branch of the revision.
245
241
             For instance, the graph {A:[], B:['A'], C:['A', 'B']} will get
249
245
             second commit in the trunk'.
250
246
         * end_of_merge: When True the next node is part of a different merge.
251
247
 
252
 
        
 
248
 
253
249
        node identifiers can be any hashable object, and are typically strings.
254
250
 
255
251
        If you have a graph like [('a', ['b']), ('a', ['c'])] this will only use
258
254
        The graph is sorted lazily: until you iterate or sort the input is
259
255
        not processed other than to create an internal representation.
260
256
 
261
 
        iteration or sorting may raise GraphCycleError if a cycle is present 
 
257
        iteration or sorting may raise GraphCycleError if a cycle is present
262
258
        in the graph.
263
259
 
264
260
        Background information on the design:
283
279
              E  1   [F]
284
280
              F 0
285
281
              C is the end of a cluster due to rule 1.
286
 
              D is not the end of a cluster from rule 1, but is from rule 2: E 
 
282
              D is not the end of a cluster from rule 1, but is from rule 2: E
287
283
                is not its left most ancestor
288
284
              E is the end of a cluster due to rule 1
289
285
              F might be but we need more data.
290
 
              
 
286
 
291
287
        we show connecting lines to a parent when:
292
288
         - The parent is the start of a merge within this cluster.
293
 
           That is, the merge was not done to the mainline before this cluster 
 
289
           That is, the merge was not done to the mainline before this cluster
294
290
           was merged to the mainline.
295
291
           This can be detected thus:
296
 
            * The parent has a higher merge depth and is the next revision in 
 
292
            * The parent has a higher merge depth and is the next revision in
297
293
              the list.
298
 
          
 
294
 
299
295
          The next revision in the list constraint is needed for this case:
300
 
          A 0   [D, B]   
301
 
          B  1  [C, F]   # we do not want to show a line to F which is depth 2 
 
296
          A 0   [D, B]
 
297
          B  1  [C, F]   # we do not want to show a line to F which is depth 2
302
298
                           but not a merge
303
 
          C  1  [H]      # note that this is a long line to show back to the 
 
299
          C  1  [H]      # note that this is a long line to show back to the
304
300
                           ancestor - see the end of merge rules.
305
301
          D 0   [G, E]
306
302
          E  1  [G, F]
341
337
        else:
342
338
            self._mainline_revisions = list(mainline_revisions)
343
339
            self._stop_revision = self._mainline_revisions[0]
344
 
        # skip the first revision, its what we reach and its parents are 
 
340
        # skip the first revision, its what we reach and its parents are
345
341
        # therefore irrelevant
346
342
        for index, revision in enumerate(self._mainline_revisions[1:]):
347
343
            # NB: index 0 means self._mainline_revisions[1]
350
346
            if parent is None:
351
347
                # end of mainline_revisions history
352
348
                continue
353
 
            if self._graph[revision][0] == parent:
 
349
            graph_parent_ids = self._graph[revision]
 
350
            if not graph_parent_ids:
 
351
                # We ran into a ghost, skip over it, this is a workaround for
 
352
                # bug #243536, the _graph has had ghosts stripped, but the
 
353
                # mainline_revisions have not
 
354
                continue
 
355
            if graph_parent_ids[0] == parent:
354
356
                continue
355
357
            # remove it from its prior spot
356
358
            self._graph[revision].remove(parent)
362
364
        self._original_graph = dict(self._graph.items())
363
365
        # we need to know the revision numbers of revisions to determine
364
366
        # the revision numbers of their descendants
365
 
        # this is a graph from node to [revno_tuple, sequence_number]
366
 
        # where sequence is the number of branches made from the node,
 
367
        # this is a graph from node to [revno_tuple, first_child]
 
368
        # where first_child is True if no other children have seen this node
367
369
        # and revno_tuple is the tuple that was assigned to the node.
368
370
        # we dont know revnos to start with, so we start it seeded with
369
 
        # [None, 0]
370
 
        self._revnos = dict((revision, [None, 0]) for revision in self._graph)
371
 
        # the global implicit root node has revno 0, but we need to know
372
 
        # the sequence number for it too:
373
 
        self._root_sequence = 0
374
 
        
 
371
        # [None, True]
 
372
        self._revnos = dict((revision, [None, True])
 
373
                            for revision in self._graph)
 
374
        # Each mainline revision counts how many child branches have spawned from it.
 
375
        self._revno_to_branch_count = {}
 
376
 
375
377
        # this is a stack storing the depth first search into the graph.
376
378
        self._node_name_stack = []
377
379
        # at each level of recursion we need the merge depth this node is at:
378
380
        self._node_merge_depth_stack = []
379
381
        # at each level of 'recursion' we have to check each parent. This
380
 
        # stack stores the parents we have not yet checked for the node at the 
 
382
        # stack stores the parents we have not yet checked for the node at the
381
383
        # matching depth in _node_name_stack
382
384
        self._pending_parents_stack = []
383
385
        # When we first look at a node we assign it a seqence number from its
384
386
        # leftmost parent.
385
 
        self._assigned_sequence_stack = []
 
387
        self._first_child_stack = []
386
388
        # this is a set of the nodes who have been completely analysed for fast
387
389
        # membership checking
388
390
        self._completed_node_names = set()
395
397
        # D 0  [F, E]
396
398
        # E  1 [F]
397
399
        # F 0
398
 
        # the scheduling order is: F, E, D, C, B, A 
 
400
        # the scheduling order is: F, E, D, C, B, A
399
401
        # that is - 'left subtree, right subtree, node'
400
402
        # which would mean that when we schedule A we can emit the entire tree.
401
403
        self._scheduled_nodes = []
402
 
        # This records for each node when we have processed its left most 
 
404
        # This records for each node when we have processed its left most
403
405
        # unmerged subtree. After this subtree is scheduled, all other subtrees
404
406
        # have their merge depth increased by one from this nodes merge depth.
405
407
        # it contains tuples - name, merge_depth
406
408
        self._left_subtree_pushed_stack = []
407
409
 
408
410
        # seed the search with the tip of the branch
409
 
        if branch_tip is not None:
 
411
        if (branch_tip is not None and
 
412
            branch_tip != _mod_revision.NULL_REVISION and
 
413
            branch_tip != (_mod_revision.NULL_REVISION,)):
410
414
            parents = self._graph.pop(branch_tip)
411
415
            self._push_node(branch_tip, 0, parents)
412
416
 
413
417
    def sorted(self):
414
418
        """Sort the graph and return as a list.
415
 
        
 
419
 
416
420
        After calling this the sorter is empty and you must create a new one.
417
421
        """
418
422
        return list(self.iter_topo_order())
419
423
 
420
424
    def iter_topo_order(self):
421
425
        """Yield the nodes of the graph in a topological order.
422
 
        
 
426
 
423
427
        After finishing iteration the sorter is empty and you cannot continue
424
428
        iteration.
425
429
        """
439
443
                      node_merge_depth_stack_append=node_merge_depth_stack.append,
440
444
                      left_subtree_pushed_stack_append=left_subtree_pushed_stack.append,
441
445
                      pending_parents_stack_append=pending_parents_stack.append,
442
 
                      assigned_sequence_stack_append=self._assigned_sequence_stack.append,
443
 
                      original_graph=self._original_graph,
 
446
                      first_child_stack_append=self._first_child_stack.append,
444
447
                      revnos=self._revnos,
445
448
                      ):
446
449
            """Add node_name to the pending node stack.
455
458
            node_merge_depth_stack_append(merge_depth)
456
459
            left_subtree_pushed_stack_append(False)
457
460
            pending_parents_stack_append(list(parents))
458
 
            # as we push it, assign it a sequence number against its parent:
459
 
            parents = original_graph[node_name]
 
461
            # as we push it, check if it is the first child
 
462
            parent_info = None
460
463
            if parents:
461
464
                # node has parents, assign from the left most parent.
462
 
                parent_revno = revnos[parents[0]]
463
 
                sequence = parent_revno[1]
464
 
                parent_revno[1] += 1
 
465
                try:
 
466
                    parent_info = revnos[parents[0]]
 
467
                except KeyError:
 
468
                    # Left-hand parent is a ghost, consider it not to exist
 
469
                    pass
 
470
            if parent_info is not None:
 
471
                first_child = parent_info[1]
 
472
                parent_info[1] = False
465
473
            else:
466
 
                # no parents, use the root sequence
467
 
                sequence = self._root_sequence
468
 
                self._root_sequence +=1
469
 
            assigned_sequence_stack_append(sequence)
 
474
                # We don't use the same algorithm here, but we need to keep the
 
475
                # stack in line
 
476
                first_child = None
 
477
            first_child_stack_append(first_child)
470
478
 
471
479
        def pop_node(node_name_stack_pop=node_name_stack.pop,
472
480
                     node_merge_depth_stack_pop=node_merge_depth_stack.pop,
473
 
                     assigned_sequence_stack_pop=self._assigned_sequence_stack.pop,
 
481
                     first_child_stack_pop=self._first_child_stack.pop,
474
482
                     left_subtree_pushed_stack_pop=left_subtree_pushed_stack.pop,
475
483
                     pending_parents_stack_pop=pending_parents_stack.pop,
476
484
                     original_graph=self._original_graph,
477
485
                     revnos=self._revnos,
478
486
                     completed_node_names_add=self._completed_node_names.add,
479
487
                     scheduled_nodes_append=scheduled_nodes.append,
 
488
                     revno_to_branch_count=self._revno_to_branch_count,
480
489
                    ):
481
490
            """Pop the top node off the stack
482
491
 
486
495
            # pop off the local variables
487
496
            node_name = node_name_stack_pop()
488
497
            merge_depth = node_merge_depth_stack_pop()
489
 
            sequence = assigned_sequence_stack_pop()
 
498
            first_child = first_child_stack_pop()
490
499
            # remove this node from the pending lists:
491
500
            left_subtree_pushed_stack_pop()
492
501
            pending_parents_stack_pop()
493
502
 
494
503
            parents = original_graph[node_name]
 
504
            parent_revno = None
495
505
            if parents:
496
506
                # node has parents, assign from the left most parent.
497
 
                parent_revno = revnos[parents[0]]
498
 
                if sequence:
 
507
                try:
 
508
                    parent_revno = revnos[parents[0]][0]
 
509
                except KeyError:
 
510
                    # left-hand parent is a ghost, treat it as not existing
 
511
                    pass
 
512
            if parent_revno is not None:
 
513
                if not first_child:
499
514
                    # not the first child, make a new branch
500
 
                    revno = parent_revno[0] + (sequence, 1)
 
515
                    base_revno = parent_revno[0]
 
516
                    branch_count = revno_to_branch_count.get(base_revno, 0)
 
517
                    branch_count += 1
 
518
                    revno_to_branch_count[base_revno] = branch_count
 
519
                    revno = (parent_revno[0], branch_count, 1)
 
520
                    # revno = (parent_revno[0], branch_count, parent_revno[-1]+1)
501
521
                else:
502
 
                    # increment the sequence number within the branch
503
 
                    revno = parent_revno[0][:-1] + (parent_revno[0][-1] + 1,)
 
522
                    # as the first child, we just increase the final revision
 
523
                    # number
 
524
                    revno = parent_revno[:-1] + (parent_revno[-1] + 1,)
504
525
            else:
505
526
                # no parents, use the root sequence
506
 
                if sequence:
507
 
                    # make a parallel import revision number
508
 
                    revno = (0, sequence, 1)
 
527
                root_count = revno_to_branch_count.get(0, -1)
 
528
                root_count += 1
 
529
                if root_count:
 
530
                    revno = (0, root_count, 1)
509
531
                else:
510
532
                    revno = (1,)
 
533
                revno_to_branch_count[0] = root_count
511
534
 
512
535
            # store the revno for this node for future reference
513
536
            revnos[node_name][0] = revno
530
553
                    if not left_subtree_pushed_stack[-1]:
531
554
                        # recurse depth first into the primary parent
532
555
                        next_node_name = pending_parents_stack[-1].pop(0)
 
556
                        is_left_subtree = True
 
557
                        left_subtree_pushed_stack[-1] = True
533
558
                    else:
534
559
                        # place any merges in right-to-left order for scheduling
535
560
                        # which gives us left-to-right order after we reverse
536
 
                        # the scheduled queue. XXX: This has the effect of 
 
561
                        # the scheduled queue. XXX: This has the effect of
537
562
                        # allocating common-new revisions to the right-most
538
 
                        # subtree rather than the left most, which will 
 
563
                        # subtree rather than the left most, which will
539
564
                        # display nicely (you get smaller trees at the top
540
565
                        # of the combined merge).
541
566
                        next_node_name = pending_parents_stack[-1].pop()
 
567
                        is_left_subtree = False
542
568
                    if next_node_name in completed_node_names:
543
569
                        # this parent was completed by a child on the
544
570
                        # call stack. skip it.
553
579
                        # current search stack (but not completed or we would
554
580
                        # have hit the continue 4 lines up.
555
581
                        # this indicates a cycle.
556
 
                        raise errors.GraphCycleError(node_name_stack)
 
582
                        if next_node_name in self._original_graph:
 
583
                            raise errors.GraphCycleError(node_name_stack)
 
584
                        else:
 
585
                            # This is just a ghost parent, ignore it
 
586
                            continue
557
587
                    next_merge_depth = 0
558
 
                    if left_subtree_pushed_stack[-1]:
 
588
                    if is_left_subtree:
559
589
                        # a new child branch from name_stack[-1]
 
590
                        next_merge_depth = 0
 
591
                    else:
560
592
                        next_merge_depth = 1
561
 
                    else:
562
 
                        next_merge_depth = 0
563
 
                        left_subtree_pushed_stack[-1] = True
564
593
                    next_merge_depth = (
565
594
                        node_merge_depth_stack[-1] + next_merge_depth)
566
595
                    push_node(
567
596
                        next_node_name,
568
597
                        next_merge_depth,
569
598
                        parents)
570
 
                    # and do not continue processing parents until this 'call' 
 
599
                    # and do not continue processing parents until this 'call'
571
600
                    # has recursed.
572
601
                    break
573
602
 
602
631
 
603
632
    def _push_node(self, node_name, merge_depth, parents):
604
633
        """Add node_name to the pending node stack.
605
 
        
 
634
 
606
635
        Names in this stack will get emitted into the output as they are popped
607
636
        off the stack.
608
637
        """
610
639
        self._node_merge_depth_stack.append(merge_depth)
611
640
        self._left_subtree_pushed_stack.append(False)
612
641
        self._pending_parents_stack.append(list(parents))
613
 
        # as we push it, assign it a sequence number against its parent:
614
 
        parents = self._original_graph[node_name]
 
642
        # as we push it, figure out if this is the first child
 
643
        parent_info = None
615
644
        if parents:
616
645
            # node has parents, assign from the left most parent.
617
 
            parent_revno = self._revnos[parents[0]]
618
 
            sequence = parent_revno[1]
619
 
            parent_revno[1] += 1
 
646
            try:
 
647
                parent_info = self._revnos[parents[0]]
 
648
            except KeyError:
 
649
                # Left-hand parent is a ghost, consider it not to exist
 
650
                pass
 
651
        if parent_info is not None:
 
652
            first_child = parent_info[1]
 
653
            parent_info[1] = False
620
654
        else:
621
 
            # no parents, use the root sequence
622
 
            sequence = self._root_sequence
623
 
            self._root_sequence +=1
624
 
        self._assigned_sequence_stack.append(sequence)
 
655
            # We don't use the same algorithm here, but we need to keep the
 
656
            # stack in line
 
657
            first_child = None
 
658
        self._first_child_stack.append(first_child)
625
659
 
626
660
    def _pop_node(self):
627
 
        """Pop the top node off the stack 
 
661
        """Pop the top node off the stack
628
662
 
629
663
        The node is appended to the sorted output.
630
664
        """
632
666
        # pop off the local variables
633
667
        node_name = self._node_name_stack.pop()
634
668
        merge_depth = self._node_merge_depth_stack.pop()
635
 
        sequence = self._assigned_sequence_stack.pop()
 
669
        first_child = self._first_child_stack.pop()
636
670
        # remove this node from the pending lists:
637
671
        self._left_subtree_pushed_stack.pop()
638
672
        self._pending_parents_stack.pop()
639
673
 
640
674
        parents = self._original_graph[node_name]
 
675
        parent_revno = None
641
676
        if parents:
642
677
            # node has parents, assign from the left most parent.
643
 
            parent_revno = self._revnos[parents[0]]
644
 
            if sequence:
 
678
            try:
 
679
                parent_revno = self._revnos[parents[0]][0]
 
680
            except KeyError:
 
681
                # left-hand parent is a ghost, treat it as not existing
 
682
                pass
 
683
        if parent_revno is not None:
 
684
            if not first_child:
645
685
                # not the first child, make a new branch
646
 
                revno = parent_revno[0] + (sequence, 1)
 
686
                base_revno = parent_revno[0]
 
687
                branch_count = self._revno_to_branch_count.get(base_revno, 0)
 
688
                branch_count += 1
 
689
                self._revno_to_branch_count[base_revno] = branch_count
 
690
                revno = (parent_revno[0], branch_count, 1)
 
691
                # revno = (parent_revno[0], branch_count, parent_revno[-1]+1)
647
692
            else:
648
 
                # increment the sequence number within the branch
649
 
                revno = parent_revno[0][:-1] + (parent_revno[0][-1] + 1,)
 
693
                # as the first child, we just increase the final revision
 
694
                # number
 
695
                revno = parent_revno[:-1] + (parent_revno[-1] + 1,)
650
696
        else:
651
697
            # no parents, use the root sequence
652
 
            if sequence:
653
 
                # make a parallel import revision number
654
 
                revno = (0, sequence, 1)
 
698
            root_count = self._revno_to_branch_count.get(0, 0)
 
699
            root_count = self._revno_to_branch_count.get(0, -1)
 
700
            root_count += 1
 
701
            if root_count:
 
702
                revno = (0, root_count, 1)
655
703
            else:
656
704
                revno = (1,)
 
705
            self._revno_to_branch_count[0] = root_count
657
706
 
658
707
        # store the revno for this node for future reference
659
708
        self._revnos[node_name][0] = revno