Skip to content

Rename MicroPather::Reset() #21

@randomMesh

Description

@randomMesh

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions