Skip to content

the toposort code is fine, but the heap equality check isn't

db48x requested to merge db48x/reposurgeon:fix-toposort into master

It checks that both sets contain the same elements, but not that they are in the same order. I've added a new method EqualWithOrdering that tests that they are precisely equal and in the same order, and used it in the toposort code. I've not checked other callers of the Equal method to see if they should be changed as well.

Merge request reports

Loading