You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When using MicroPather in a modern application using std::unique_ptr, the Reset method could easily be confused with the reset method of the smart pointer.
std::unique_ptr<micropather::MicroPather> pather;
// costs between states changed
pather->Reset();
// delete the pointer explicitly
pather.reset();
I therefore suggest to rename the Reset method to StateCostChanged or similar.
You should keep Reset and use it as a wrapper for the new method in order to keep backwards compatibility, though.
The text was updated successfully, but these errors were encountered:
When using MicroPather in a modern application using std::unique_ptr, the
Reset
method could easily be confused with thereset
method of the smart pointer.I therefore suggest to rename the
Reset
method toStateCostChanged
or similar.You should keep
Reset
and use it as a wrapper for the new method in order to keep backwards compatibility, though.The text was updated successfully, but these errors were encountered: