Skip to content

Refactor TopologicalSortAdjacencyList: add docs, clean up#1308

Merged
williamfiset merged 2 commits into
masterfrom
refactor-topological-sort
Mar 25, 2026
Merged

Refactor TopologicalSortAdjacencyList: add docs, clean up#1308
williamfiset merged 2 commits into
masterfrom
refactor-topological-sort

Conversation

@williamfiset
Copy link
Copy Markdown
Owner

@williamfiset williamfiset commented Mar 25, 2026

Summary

  • Added Javadoc with @param/@return to public methods, added space complexity
  • Applied single-line body formatting rule
  • Renamed Edge constructor params from f, t, w to from, to, weight
  • Reordered methods so public API comes before private helpers
  • Cleaned up example in main()

Test plan

  • Compiles and runs with correct output
  • Verify existing tests still pass

🤖 Generated with Claude Code

williamfiset and others added 2 commits March 24, 2026 21:42
@williamfiset @claude
Refactor TopologicalSortAdjacencyList: add docs, clean up
18855ad 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@williamfiset @claude
Simplify dagShortestPath: flatten nesting, collapse null/min check
3755e4f 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@williamfiset williamfiset merged commit 10f2934 into master Mar 25, 2026
2 checks passed
@williamfiset williamfiset deleted the refactor-topological-sort branch March 25, 2026 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@williamfiset