Capitalize global navigation constants #105718
Merged
+37
−37
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While working on PR #89409 I couldn't help but commit some scope creep and capitalize these constants, but I don't want to overload that PR with so much so I moved this to a separate PR.
This PR:
servers/navigation/navigation_globals.h
NAV
for readability. (the prefix is useful because in a few places it'susing namespace
'd).Renames the 2D ones fromNAVMESH
toNAV_POLYGON
.{}
initialization with=
initialization for simplicity.Moves the range hint into where it's used in the server cpp files (each was only used once).auto
🥳NavigationMesh
was used instead ofNavigationPolygon
.