New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pathfinder upgrade #126
Pathfinder upgrade #126
Conversation
Great! I updated a few details in #106 yesterday, all regarding to the various npc bumping aspects. |
gemrb/core/PathFinder.cpp
Outdated
// target (the goal must be in sight of the end, if PF_SIGHT is specified) | ||
PathNode *Map::FindPath(const Point &s, const Point &d, unsigned int size, unsigned int minDistance, int flags, const Actor *caller) const | ||
{ | ||
Log(DEBUG, "FindPath", "s = (%d, %d), d = (%d, %d), caller = %s, dist = %d, size = %d", s.x, s.y, d.x, d.y, caller ? caller->GetName(0) : "NULL", minDistance, size); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Time to go or should we wait a bit?
Log(DEBUG, "FindPath", "s = (%d, %d), d = (%d, %d), caller = %s, dist = %d, size = %d", s.x, s.y, d.x, d.y, caller ? caller->GetName(0) : "NULL", minDistance, size); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is less frequent. I think we can leave it in for now, and if somebody complains about a spam of these it's either an issue or we can just remove it then. If nobody complains about it for, say, a month, then I guess it can go.
Here is an overview of what got changed by this pull request: Issues
======
+ Solved 2
Clones removed
==============
+ gemrb/core/Map.cpp -15
See the complete overview on Codacy |
Thanks for taking this long path with me! 🎆 |
I hope these commits find their new home comfortable! ❄️(sorry, had to do this) |
Checklist