Skip to content
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

Use shorter scale specifications in project_to, etc. #42317

Merged
merged 1 commit into from
Jul 21, 2020

Conversation

jbytheway
Copy link
Contributor

@jbytheway jbytheway commented Jul 20, 2020

Summary

SUMMARY: None

Purpose of change

When working on #41693 I totally forgot that I'd added shorter names for the various coord::scale values to use in project_to etc. without being so verbose.

This makes the API less verbose and more consistent because the names of the scales match the abbreviations in the type names.

Describe the solution

Convert to the shorter names.

Strictly a find-and-replace commit, so hopefully should be safe.

Describe alternatives you've considered

Leaving the long names for more clarity.

Testing

Unit tests.

When working on CleverRaven#41693 I totally forgot that I'd added shorter names for
the various coord::scale values to use in project_to etc. without being
so verbose.

This makes the API more consistent because the names of the scales match
the abbreviations in the type names.

Convert to the shorter names.

Strictly a find-and-replace commit, so hopefully should be safe.
@jbytheway jbytheway requested a review from KorGgenT as a code owner July 20, 2020 21:02
@ZhilkinSerg ZhilkinSerg added [C++] Changes (can be) made in C++. Previously named `Code` Code: Infrastructure / Style / Static Analysis Code internal infrastructure and style labels Jul 21, 2020
@kevingranade kevingranade merged commit ab7a370 into CleverRaven:master Jul 21, 2020
@jbytheway jbytheway deleted the use_shorter_scale_names branch July 21, 2020 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Code: Infrastructure / Style / Static Analysis Code internal infrastructure and style
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants