Skip to content

Modify ClosestPoint argument order #27

@dfridovi

Description

@dfridovi

Would make sense to have this argument (is_endpoint in the ClosestPoint() function in Polyline2) appear after is_vertex to minimize chances of misuse. that would mean changing a bunch of other uses where this appears, but I think that's the right way to do it long term. Refer to:
https://github.com/HJReachability/ilqgames/pull/24/files#r418777414

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions