~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tsort.py

  • Committer: Martin Pool
  • Date: 2010-01-29 10:36:23 UTC
  • mto: This revision was merged to the branch mainline in revision 4992.
  • Revision ID: mbp@sourcefrog.net-20100129103623-hywka5hymo5z13jw
Change url to canonical.com or wiki, plus some doc improvements in passing

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
 
import bzrlib.errors as 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
 
        ### if debugging:
65
 
        # self._original_graph = dict(graph)
66
 
        
67
 
        # this is a stack storing the depth first search into the graph.
68
 
        self._node_name_stack = []
69
 
        # at each level of 'recursion' we have to check each parent. This
70
 
        # stack stores the parents we have not yet checked for the node at the 
71
 
        # matching depth in _node_name_stack
72
 
        self._pending_parents_stack = []
73
 
        # this is a set of the completed nodes for fast checking whether a
74
 
        # parent in a node we are processing on the stack has already been
75
 
        # emitted and thus can be skipped.
76
 
        self._completed_node_names = set()
77
77
 
78
78
    def sorted(self):
79
79
        """Sort the graph and return as a list.
80
 
        
 
80
 
81
81
        After calling this the sorter is empty and you must create a new one.
82
82
        """
83
83
        return list(self.iter_topo_order())
94
94
 
95
95
    def iter_topo_order(self):
96
96
        """Yield the nodes of the graph in a topological order.
97
 
        
 
97
 
98
98
        After finishing iteration the sorter is empty and you cannot continue
99
99
        iteration.
100
100
        """
101
 
        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:
102
117
            # now pick a random node in the source graph, and transfer it to the
103
 
            # top of the depth first search stack.
104
 
            node_name, parents = self._graph.popitem()
105
 
            self._push_node(node_name, parents)
106
 
            while self._node_name_stack:
107
 
                # loop until this call completes.
108
 
                parents_to_visit = self._pending_parents_stack[-1]
109
 
                # 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
110
127
                if not parents_to_visit:
111
128
                    # append the revision to the topo sorted list
112
 
                    # all the nodes parents have been added to the output, now
113
 
                    # we can add it to the output.
114
 
                    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
115
135
                else:
116
 
                    while self._pending_parents_stack[-1]:
117
 
                        # recurse depth first into a single parent 
118
 
                        next_node_name = self._pending_parents_stack[-1].pop()
119
 
                        if next_node_name in self._completed_node_names:
120
 
                            # this parent was completed by a child on the
121
 
                            # call stack. skip it.
122
 
                            continue
123
 
                        # otherwise transfer it from the source graph into the
124
 
                        # top of the current depth first search stack.
125
 
                        try:
126
 
                            parents = self._graph.pop(next_node_name)
127
 
                        except KeyError:
128
 
                            # if the next node is not in the source graph it has
129
 
                            # already been popped from it and placed into the
130
 
                            # current search stack (but not completed or we would
131
 
                            # have hit the continue 4 lines up.
132
 
                            # this indicates a cycle.
133
 
                            raise errors.GraphCycleError(self._node_name_stack)
134
 
                        self._push_node(next_node_name, parents)
135
 
                        # and do not continue processing parents until this 'call' 
136
 
                        # has recursed.
137
 
                        break
138
 
 
139
 
    def _push_node(self, node_name, parents):
140
 
        """Add node_name to the pending node stack.
141
 
        
142
 
        Names in this stack will get emitted into the output as they are popped
143
 
        off the stack.
144
 
        """
145
 
        self._node_name_stack.append(node_name)
146
 
        self._pending_parents_stack.append(list(parents))
147
 
 
148
 
    def _pop_node(self):
149
 
        """Pop the top node off the stack 
150
 
 
151
 
        The node is appended to the sorted output.
152
 
        """
153
 
        # we are returning from the flattened call frame:
154
 
        # pop off the local variables
155
 
        node_name = self._node_name_stack.pop()
156
 
        self._pending_parents_stack.pop()
157
 
 
158
 
        self._completed_node_names.add(node_name)
159
 
        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))
160
159
 
161
160
 
162
161
def merge_sort(graph, branch_tip, mainline_revisions=None, generate_revno=False):
163
162
    """Topological sort a graph which groups merges.
164
163
 
165
164
    :param graph: sequence of pairs of node->parents_list.
166
 
    :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
167
166
                       reachable from branch_tip are not included in the
168
167
                       output.
169
168
    :param mainline_revisions: If not None this forces a mainline to be
186
185
 
187
186
class MergeSorter(object):
188
187
 
 
188
    __slots__ = ['_node_name_stack',
 
189
                 '_node_merge_depth_stack',
 
190
                 '_pending_parents_stack',
 
191
                 '_first_child_stack',
 
192
                 '_left_subtree_pushed_stack',
 
193
                 '_generate_revno',
 
194
                 '_graph',
 
195
                 '_mainline_revisions',
 
196
                 '_stop_revision',
 
197
                 '_original_graph',
 
198
                 '_revnos',
 
199
                 '_revno_to_branch_count',
 
200
                 '_completed_node_names',
 
201
                 '_scheduled_nodes',
 
202
                ]
 
203
 
189
204
    def __init__(self, graph, branch_tip, mainline_revisions=None,
190
205
        generate_revno=False):
191
206
        """Merge-aware topological sorting of a graph.
192
 
    
 
207
 
193
208
        :param graph: sequence of pairs of node_name->parent_names_list.
194
209
                      i.e. [('C', ['B']), ('B', ['A']), ('A', [])]
195
210
                      For this input the output from the sort or
196
211
                      iter_topo_order routines will be:
197
212
                      'A', 'B', 'C'
198
 
        :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
199
214
                       reachable from branch_tip are not included in the
200
215
                       output.
201
216
        :param mainline_revisions: If not None this forces a mainline to be
213
228
        The result is a list sorted so that all parents come before
214
229
        their children. Each element of the list is a tuple containing:
215
230
        (sequence_number, node_name, merge_depth, end_of_merge)
216
 
         * 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
217
232
           GUIs.
218
233
         * node_name: The node name: opaque text to the merge routine.
219
234
         * merge_depth: How many levels of merging deep this node has been
220
235
           found.
221
236
         * revno_sequence: When requested this field provides a sequence of
222
237
             revision numbers for all revisions. The format is:
223
 
             REVNO[[.BRANCHREVNO.REVNO] ...]. BRANCHREVNO is the number of the
 
238
             (REVNO, BRANCHNUM, BRANCHREVNO). BRANCHNUM is the number of the
224
239
             branch that the revno is on. From left to right the REVNO numbers
225
240
             are the sequence numbers within that branch of the revision.
226
241
             For instance, the graph {A:[], B:['A'], C:['A', 'B']} will get
230
245
             second commit in the trunk'.
231
246
         * end_of_merge: When True the next node is part of a different merge.
232
247
 
233
 
        
 
248
 
234
249
        node identifiers can be any hashable object, and are typically strings.
235
250
 
236
251
        If you have a graph like [('a', ['b']), ('a', ['c'])] this will only use
239
254
        The graph is sorted lazily: until you iterate or sort the input is
240
255
        not processed other than to create an internal representation.
241
256
 
242
 
        iteration or sorting may raise GraphCycleError if a cycle is present 
 
257
        iteration or sorting may raise GraphCycleError if a cycle is present
243
258
        in the graph.
244
259
 
245
260
        Background information on the design:
264
279
              E  1   [F]
265
280
              F 0
266
281
              C is the end of a cluster due to rule 1.
267
 
              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
268
283
                is not its left most ancestor
269
284
              E is the end of a cluster due to rule 1
270
285
              F might be but we need more data.
271
 
              
 
286
 
272
287
        we show connecting lines to a parent when:
273
288
         - The parent is the start of a merge within this cluster.
274
 
           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
275
290
           was merged to the mainline.
276
291
           This can be detected thus:
277
 
            * 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
278
293
              the list.
279
 
          
 
294
 
280
295
          The next revision in the list constraint is needed for this case:
281
 
          A 0   [D, B]   
282
 
          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
283
298
                           but not a merge
284
 
          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
285
300
                           ancestor - see the end of merge rules.
286
301
          D 0   [G, E]
287
302
          E  1  [G, F]
322
337
        else:
323
338
            self._mainline_revisions = list(mainline_revisions)
324
339
            self._stop_revision = self._mainline_revisions[0]
325
 
        # 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
326
341
        # therefore irrelevant
327
342
        for index, revision in enumerate(self._mainline_revisions[1:]):
328
343
            # NB: index 0 means self._mainline_revisions[1]
331
346
            if parent is None:
332
347
                # end of mainline_revisions history
333
348
                continue
334
 
            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:
335
356
                continue
336
357
            # remove it from its prior spot
337
358
            self._graph[revision].remove(parent)
343
364
        self._original_graph = dict(self._graph.items())
344
365
        # we need to know the revision numbers of revisions to determine
345
366
        # the revision numbers of their descendants
346
 
        # this is a graph from node to [revno_tuple, sequence_number]
347
 
        # 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
348
369
        # and revno_tuple is the tuple that was assigned to the node.
349
370
        # we dont know revnos to start with, so we start it seeded with
350
 
        # [None, 0]
351
 
        self._revnos = dict((revision, [None, 0]) for revision in self._graph)
352
 
        # the global implicit root node has revno 0, but we need to know
353
 
        # the sequence number for it too:
354
 
        self._root_sequence = 0
355
 
        
 
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
 
356
377
        # this is a stack storing the depth first search into the graph.
357
378
        self._node_name_stack = []
358
379
        # at each level of recursion we need the merge depth this node is at:
359
380
        self._node_merge_depth_stack = []
360
381
        # at each level of 'recursion' we have to check each parent. This
361
 
        # 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
362
383
        # matching depth in _node_name_stack
363
384
        self._pending_parents_stack = []
364
385
        # When we first look at a node we assign it a seqence number from its
365
386
        # leftmost parent.
366
 
        self._assigned_sequence_stack = []
 
387
        self._first_child_stack = []
367
388
        # this is a set of the nodes who have been completely analysed for fast
368
389
        # membership checking
369
390
        self._completed_node_names = set()
376
397
        # D 0  [F, E]
377
398
        # E  1 [F]
378
399
        # F 0
379
 
        # the scheduling order is: F, E, D, C, B, A 
 
400
        # the scheduling order is: F, E, D, C, B, A
380
401
        # that is - 'left subtree, right subtree, node'
381
402
        # which would mean that when we schedule A we can emit the entire tree.
382
403
        self._scheduled_nodes = []
383
 
        # 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
384
405
        # unmerged subtree. After this subtree is scheduled, all other subtrees
385
406
        # have their merge depth increased by one from this nodes merge depth.
386
407
        # it contains tuples - name, merge_depth
387
408
        self._left_subtree_pushed_stack = []
388
409
 
389
410
        # seed the search with the tip of the branch
390
 
        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,)):
391
414
            parents = self._graph.pop(branch_tip)
392
415
            self._push_node(branch_tip, 0, parents)
393
416
 
394
417
    def sorted(self):
395
418
        """Sort the graph and return as a list.
396
 
        
 
419
 
397
420
        After calling this the sorter is empty and you must create a new one.
398
421
        """
399
422
        return list(self.iter_topo_order())
400
423
 
401
424
    def iter_topo_order(self):
402
425
        """Yield the nodes of the graph in a topological order.
403
 
        
 
426
 
404
427
        After finishing iteration the sorter is empty and you cannot continue
405
428
        iteration.
406
429
        """
407
 
        while self._node_name_stack:
 
430
        # These are safe to offload to local variables, because they are used
 
431
        # as a stack and modified in place, never assigned to.
 
432
        node_name_stack = self._node_name_stack
 
433
        node_merge_depth_stack = self._node_merge_depth_stack
 
434
        pending_parents_stack = self._pending_parents_stack
 
435
        left_subtree_pushed_stack = self._left_subtree_pushed_stack
 
436
        completed_node_names = self._completed_node_names
 
437
        scheduled_nodes = self._scheduled_nodes
 
438
 
 
439
        graph_pop = self._graph.pop
 
440
 
 
441
        def push_node(node_name, merge_depth, parents,
 
442
                      node_name_stack_append=node_name_stack.append,
 
443
                      node_merge_depth_stack_append=node_merge_depth_stack.append,
 
444
                      left_subtree_pushed_stack_append=left_subtree_pushed_stack.append,
 
445
                      pending_parents_stack_append=pending_parents_stack.append,
 
446
                      first_child_stack_append=self._first_child_stack.append,
 
447
                      revnos=self._revnos,
 
448
                      ):
 
449
            """Add node_name to the pending node stack.
 
450
 
 
451
            Names in this stack will get emitted into the output as they are popped
 
452
            off the stack.
 
453
 
 
454
            This inlines a lot of self._variable.append functions as local
 
455
            variables.
 
456
            """
 
457
            node_name_stack_append(node_name)
 
458
            node_merge_depth_stack_append(merge_depth)
 
459
            left_subtree_pushed_stack_append(False)
 
460
            pending_parents_stack_append(list(parents))
 
461
            # as we push it, check if it is the first child
 
462
            parent_info = None
 
463
            if parents:
 
464
                # node has parents, assign from the left most parent.
 
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
 
473
            else:
 
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)
 
478
 
 
479
        def pop_node(node_name_stack_pop=node_name_stack.pop,
 
480
                     node_merge_depth_stack_pop=node_merge_depth_stack.pop,
 
481
                     first_child_stack_pop=self._first_child_stack.pop,
 
482
                     left_subtree_pushed_stack_pop=left_subtree_pushed_stack.pop,
 
483
                     pending_parents_stack_pop=pending_parents_stack.pop,
 
484
                     original_graph=self._original_graph,
 
485
                     revnos=self._revnos,
 
486
                     completed_node_names_add=self._completed_node_names.add,
 
487
                     scheduled_nodes_append=scheduled_nodes.append,
 
488
                     revno_to_branch_count=self._revno_to_branch_count,
 
489
                    ):
 
490
            """Pop the top node off the stack
 
491
 
 
492
            The node is appended to the sorted output.
 
493
            """
 
494
            # we are returning from the flattened call frame:
 
495
            # pop off the local variables
 
496
            node_name = node_name_stack_pop()
 
497
            merge_depth = node_merge_depth_stack_pop()
 
498
            first_child = first_child_stack_pop()
 
499
            # remove this node from the pending lists:
 
500
            left_subtree_pushed_stack_pop()
 
501
            pending_parents_stack_pop()
 
502
 
 
503
            parents = original_graph[node_name]
 
504
            parent_revno = None
 
505
            if parents:
 
506
                # node has parents, assign from the left most parent.
 
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:
 
514
                    # not the first child, make a new branch
 
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)
 
521
                else:
 
522
                    # as the first child, we just increase the final revision
 
523
                    # number
 
524
                    revno = parent_revno[:-1] + (parent_revno[-1] + 1,)
 
525
            else:
 
526
                # no parents, use the root sequence
 
527
                root_count = revno_to_branch_count.get(0, -1)
 
528
                root_count += 1
 
529
                if root_count:
 
530
                    revno = (0, root_count, 1)
 
531
                else:
 
532
                    revno = (1,)
 
533
                revno_to_branch_count[0] = root_count
 
534
 
 
535
            # store the revno for this node for future reference
 
536
            revnos[node_name][0] = revno
 
537
            completed_node_names_add(node_name)
 
538
            scheduled_nodes_append((node_name, merge_depth, revno))
 
539
            return node_name
 
540
 
 
541
 
 
542
        while node_name_stack:
408
543
            # loop until this call completes.
409
 
            parents_to_visit = self._pending_parents_stack[-1]
 
544
            parents_to_visit = pending_parents_stack[-1]
410
545
            # if all parents are done, the revision is done
411
546
            if not parents_to_visit:
412
547
                # append the revision to the topo sorted scheduled list:
413
548
                # all the nodes parents have been scheduled added, now
414
549
                # we can add it to the output.
415
 
                self._pop_node()
 
550
                pop_node()
416
551
            else:
417
 
                while self._pending_parents_stack[-1]:
418
 
                    if not self._left_subtree_pushed_stack[-1]:
 
552
                while pending_parents_stack[-1]:
 
553
                    if not left_subtree_pushed_stack[-1]:
419
554
                        # recurse depth first into the primary parent
420
 
                        next_node_name = self._pending_parents_stack[-1].pop(0)
 
555
                        next_node_name = pending_parents_stack[-1].pop(0)
 
556
                        is_left_subtree = True
 
557
                        left_subtree_pushed_stack[-1] = True
421
558
                    else:
422
559
                        # place any merges in right-to-left order for scheduling
423
560
                        # which gives us left-to-right order after we reverse
424
 
                        # the scheduled queue. XXX: This has the effect of 
 
561
                        # the scheduled queue. XXX: This has the effect of
425
562
                        # allocating common-new revisions to the right-most
426
 
                        # subtree rather than the left most, which will 
 
563
                        # subtree rather than the left most, which will
427
564
                        # display nicely (you get smaller trees at the top
428
565
                        # of the combined merge).
429
 
                        next_node_name = self._pending_parents_stack[-1].pop()
430
 
                    if next_node_name in self._completed_node_names:
 
566
                        next_node_name = pending_parents_stack[-1].pop()
 
567
                        is_left_subtree = False
 
568
                    if next_node_name in completed_node_names:
431
569
                        # this parent was completed by a child on the
432
570
                        # call stack. skip it.
433
571
                        continue
434
572
                    # otherwise transfer it from the source graph into the
435
573
                    # top of the current depth first search stack.
436
574
                    try:
437
 
                        parents = self._graph.pop(next_node_name)
 
575
                        parents = graph_pop(next_node_name)
438
576
                    except KeyError:
439
577
                        # if the next node is not in the source graph it has
440
578
                        # already been popped from it and placed into the
441
579
                        # current search stack (but not completed or we would
442
580
                        # have hit the continue 4 lines up.
443
581
                        # this indicates a cycle.
444
 
                        raise errors.GraphCycleError(self._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
445
587
                    next_merge_depth = 0
446
 
                    if self._left_subtree_pushed_stack[-1]:
 
588
                    if is_left_subtree:
447
589
                        # a new child branch from name_stack[-1]
 
590
                        next_merge_depth = 0
 
591
                    else:
448
592
                        next_merge_depth = 1
449
 
                    else:
450
 
                        next_merge_depth = 0
451
 
                        self._left_subtree_pushed_stack[-1] = True
452
593
                    next_merge_depth = (
453
 
                        self._node_merge_depth_stack[-1] + next_merge_depth)
454
 
                    self._push_node(
 
594
                        node_merge_depth_stack[-1] + next_merge_depth)
 
595
                    push_node(
455
596
                        next_node_name,
456
597
                        next_merge_depth,
457
598
                        parents)
458
 
                    # and do not continue processing parents until this 'call' 
 
599
                    # and do not continue processing parents until this 'call'
459
600
                    # has recursed.
460
601
                    break
 
602
 
461
603
        # We have scheduled the graph. Now deliver the ordered output:
462
604
        sequence_number = 0
463
 
        while self._scheduled_nodes:
464
 
            node_name, merge_depth, revno = self._scheduled_nodes.pop()
465
 
            if node_name == self._stop_revision:
 
605
        stop_revision = self._stop_revision
 
606
        generate_revno = self._generate_revno
 
607
        original_graph = self._original_graph
 
608
 
 
609
        while scheduled_nodes:
 
610
            node_name, merge_depth, revno = scheduled_nodes.pop()
 
611
            if node_name == stop_revision:
466
612
                return
467
 
            if not len(self._scheduled_nodes):
 
613
            if not len(scheduled_nodes):
468
614
                # last revision is the end of a merge
469
615
                end_of_merge = True
470
 
            elif self._scheduled_nodes[-1][1] < merge_depth:
 
616
            elif scheduled_nodes[-1][1] < merge_depth:
471
617
                # the next node is to our left
472
618
                end_of_merge = True
473
 
            elif (self._scheduled_nodes[-1][1] == merge_depth and
474
 
                  (self._scheduled_nodes[-1][0] not in
475
 
                   self._original_graph[node_name])):
 
619
            elif (scheduled_nodes[-1][1] == merge_depth and
 
620
                  (scheduled_nodes[-1][0] not in
 
621
                   original_graph[node_name])):
476
622
                # the next node was part of a multiple-merge.
477
623
                end_of_merge = True
478
624
            else:
479
625
                end_of_merge = False
480
 
            if self._generate_revno:
 
626
            if generate_revno:
481
627
                yield (sequence_number, node_name, merge_depth, revno, end_of_merge)
482
628
            else:
483
629
                yield (sequence_number, node_name, merge_depth, end_of_merge)
485
631
 
486
632
    def _push_node(self, node_name, merge_depth, parents):
487
633
        """Add node_name to the pending node stack.
488
 
        
 
634
 
489
635
        Names in this stack will get emitted into the output as they are popped
490
636
        off the stack.
491
637
        """
493
639
        self._node_merge_depth_stack.append(merge_depth)
494
640
        self._left_subtree_pushed_stack.append(False)
495
641
        self._pending_parents_stack.append(list(parents))
496
 
        # as we push it, assign it a sequence number against its parent:
497
 
        parents = self._original_graph[node_name]
 
642
        # as we push it, figure out if this is the first child
 
643
        parent_info = None
498
644
        if parents:
499
645
            # node has parents, assign from the left most parent.
500
 
            parent_revno = self._revnos[parents[0]]
501
 
            sequence = parent_revno[1]
502
 
            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
503
654
        else:
504
 
            # no parents, use the root sequence
505
 
            sequence = self._root_sequence
506
 
            self._root_sequence +=1
507
 
        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)
508
659
 
509
660
    def _pop_node(self):
510
 
        """Pop the top node off the stack 
 
661
        """Pop the top node off the stack
511
662
 
512
663
        The node is appended to the sorted output.
513
664
        """
515
666
        # pop off the local variables
516
667
        node_name = self._node_name_stack.pop()
517
668
        merge_depth = self._node_merge_depth_stack.pop()
518
 
        sequence = self._assigned_sequence_stack.pop()
 
669
        first_child = self._first_child_stack.pop()
519
670
        # remove this node from the pending lists:
520
671
        self._left_subtree_pushed_stack.pop()
521
672
        self._pending_parents_stack.pop()
522
673
 
523
674
        parents = self._original_graph[node_name]
 
675
        parent_revno = None
524
676
        if parents:
525
677
            # node has parents, assign from the left most parent.
526
 
            parent_revno = self._revnos[parents[0]]
527
 
            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:
528
685
                # not the first child, make a new branch
529
 
                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)
530
692
            else:
531
 
                # increment the sequence number within the branch
532
 
                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,)
533
696
        else:
534
697
            # no parents, use the root sequence
535
 
            if sequence:
536
 
                # make a parallel import revision number
537
 
                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)
538
703
            else:
539
704
                revno = (1,)
 
705
            self._revno_to_branch_count[0] = root_count
540
706
 
541
707
        # store the revno for this node for future reference
542
708
        self._revnos[node_name][0] = revno