RogueSharp icon indicating copy to clipboard operation
RogueSharp copied to clipboard

Methods that return paths should be consistent with inclusion of source and destination

Open FaronBracy opened this issue 6 years ago • 0 comments

The DepthFirstPaths class has a PathTo method that returns a list of vertices from source to destination https://github.com/FaronBracy/RogueSharp/blob/master/RogueSharp/PathFinder.cs#L100 The source vertex is not included in the list.

This behavior is different from the PathFinder class's ShortestPath method which does include the source cell. https://github.com/FaronBracy/RogueSharp/blob/master/RogueSharp/PathFinder.cs#L100

The behavior should be consistent across all methods that return paths

FaronBracy avatar Feb 02 '19 17:02 FaronBracy