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

native_window: Remove redundant TRANSFORM_ prefix from NativeWindowTransform variants #460

Merged
merged 1 commit into from
Jan 29, 2024

Conversation

MarijnS95
Copy link
Member

This prefix is already included in the type name, when stripping down the "typename" ANATIVEWINDOW_TRANSFORM_ prefix from every enum constant.

@MarijnS95 MarijnS95 force-pushed the native-window-remove-transform-prefix branch 2 times, most recently from 50e9063 to 3b12db7 Compare December 25, 2023 22:39
@MarijnS95 MarijnS95 changed the title native_window: Remove redundant TRANSFORM_ prefix from `NativeWindowTransform variants native_window: Remove redundant TRANSFORM_ prefix from NativeWindowTransform variants Dec 25, 2023
@MarijnS95 MarijnS95 force-pushed the native-window-remove-transform-prefix branch from 3b12db7 to 5a982d8 Compare December 25, 2023 22:39
@MarijnS95 MarijnS95 added the impact: breaking API/ABI-breaking change label Dec 25, 2023
@MarijnS95 MarijnS95 requested a review from rib January 18, 2024 22:03
…indowTransform` variants

This prefix is already included in the type name, when stripping
down the "typename" `ANATIVEWINDOW_TRANSFORM_` prefix from every enum
constant.
@MarijnS95 MarijnS95 force-pushed the native-window-remove-transform-prefix branch from 5a982d8 to 509dcb6 Compare January 27, 2024 22:05
@MarijnS95 MarijnS95 merged commit a16f995 into master Jan 29, 2024
38 checks passed
@MarijnS95 MarijnS95 deleted the native-window-remove-transform-prefix branch January 29, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact: breaking API/ABI-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants