Refactor path handling - Githubissues

Skretzo / shortest-path

Pathfinding for Old School RuneScape

BSD 2-Clause "Simplified" License

14 stars 31 forks source link

Refactor path handling #7

Closed cyberlarino closed 2 years ago

cyberlarino commented 2 years ago

tried to clean paths handling up a bit;

Path class used to be the task which calculated the entire path. Added a struct only for keeping path named "Path", and moved all this functionality to "PathfinderTask", to better reflect what was going on.
added OrdinalDirections enum because of DRY

Skretzo commented 2 years ago

This is unfortunately not very obvious, but the live version of the plugin is currently not the master branch. I will quickly deal with this, and then you will probably need to resolve conflicts.

Skretzo commented 2 years ago

You will need to redirect this PR to the master branch.

cyberlarino commented 2 years ago

This is unfortunately not very obvious, but the live version of the plugin is currently not the master branch. I will quickly deal with this, and then you will probably need to resolve conflicts.

Ah that's unfortunate. I will have a look.

cyberlarino commented 2 years ago

Obsolete by #11