RogueSharp
RogueSharp copied to clipboard
Methods that return IEnumerable<T> should be consistent with yields
DijkstraShortestPath.PathTo method returns an IEnumerable and could yield return DirectedEdge. https://github.com/FaronBracy/RogueSharp/blob/master/RogueSharp/Algorithms/DijkstraShortestPath.cs#L130
Currently it does not, yet many other methods in the solution that return IEnumerable do. https://github.com/FaronBracy/RogueSharp/blob/master/RogueSharp/Map.cs#L293
Investigate which approach is better and be consistent within the project.