Skip to content

Commit 7ccab5d

Browse files
hjothamendixclaude
andcommitted
chore: remove dead collectReachableNodes helper
The describer no longer references collectReachableNodes; selectNearestCommonMerge uses collectReachableDistances exclusively. Drop the unused helper to keep the file focused. Addresses ako's cleanup note on PR #327. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 46ef628 commit 7ccab5d

1 file changed

Lines changed: 0 additions & 27 deletions

File tree

mdl/executor/cmd_microflows_show.go

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -889,33 +889,6 @@ func findMergeForSplit(
889889
return selectNearestCommonJoin(activityMap, branchDistances)
890890
}
891891

892-
// collectReachableNodes collects all nodes reachable from a starting node.
893-
func collectReachableNodes(
894-
ctx *ExecContext,
895-
startID model.ID,
896-
flowsByOrigin map[model.ID][]*microflows.SequenceFlow,
897-
activityMap map[model.ID]microflows.MicroflowObject,
898-
visited map[model.ID]bool,
899-
) map[model.ID]bool {
900-
result := make(map[model.ID]bool)
901-
902-
var traverse func(id model.ID)
903-
traverse = func(id model.ID) {
904-
if visited[id] {
905-
return
906-
}
907-
visited[id] = true
908-
result[id] = true
909-
910-
for _, flow := range flowsByOrigin[id] {
911-
traverse(flow.DestinationID)
912-
}
913-
}
914-
915-
traverse(startID)
916-
return result
917-
}
918-
919892
// collectReachableDistances collects the shortest normal-flow distance from a
920893
// branch start to every reachable node. Error handler flows are excluded because
921894
// they do not participate in split/merge structural pairing.

0 commit comments

Comments
 (0)