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

C++ style enums 7/N: MeasureMode #39452

Closed
wants to merge 7 commits into from

Conversation

NickGerleman
Copy link
Contributor

Summary:
X-link: facebook/yoga#1389

This converts usages of YGMeasureMode to MeasureMode

Differential Revision: D49271165

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Sep 14, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49271165

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 14, 2023
Summary:
X-link: facebook/react-native#39452

Pull Request resolved: facebook#1389

This converts usages of YGMeasureMode to MeasureMode

Differential Revision: D49271165

fbshipit-source-id: 2ea493123db9a9ca1e47454a39ba6838bbff98e8
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49271165

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 14, 2023
Summary:
Pull Request resolved: facebook#39452

X-link: facebook/yoga#1389

This converts usages of YGMeasureMode to MeasureMode

Differential Revision: D49271165

fbshipit-source-id: 84b79aed61cc34f91292911e0be2663c3cf481cc
@analysis-bot
Copy link

analysis-bot commented Sep 14, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,329,398 +153
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 9,568,155 +145
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 598b7ed
Branch: main

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49271165

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 15, 2023
Summary:
X-link: facebook/react-native#39452

Pull Request resolved: facebook#1389

This converts usages of YGMeasureMode to MeasureMode

Reviewed By: rozele

Differential Revision: D49271165

fbshipit-source-id: 458562c5944e3574cd86fbbc9c5890076ddce7da
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 15, 2023
Summary:
Pull Request resolved: facebook#39452

X-link: facebook/yoga#1389

This converts usages of YGMeasureMode to MeasureMode

Reviewed By: rozele

Differential Revision: D49271165

fbshipit-source-id: a9646ed7d625471064953e44ad3fe18725dfac84
NickGerleman and others added 7 commits September 14, 2023 20:50
Summary:
This adds logic to the enum generator to generate C++ style scoped enums.

This gives us a few nicities over C enums, even if both must exist:
1. We can add types and keep unsgined enums directly in bitfields
2. Style/readability
3. Avoiding implicit int conversion

Differential Revision: D49267996

fbshipit-source-id: bfde22f7361c9160b9c475fa458b3dc89c9fab19
Differential Revision: D49269117

fbshipit-source-id: 43d4aaad8ee30a8150f0edc9f8a445e2b2e6f40a
Differential Revision: D49269440

fbshipit-source-id: 628a91fda1db878a531df47359f08deb0cc65e00
Summary: This converts usages of YGErrata to Errata

Differential Revision: D49270354

fbshipit-source-id: 6749b71a3f3c32e34d58265097962153b50e87f6
Summary: This converts usages of YGLogLevel to LogLevel

Differential Revision: D49270695

fbshipit-source-id: 2c9d3dbd2002af1a48fce704a5ab0be4147293b7
Summary: This converts usages of YGPrintOptions to PrintOptions

Differential Revision: D49270929

fbshipit-source-id: 532996d68188ee7709b1a72fbde34a5816de0d16
Summary:
Pull Request resolved: facebook#39452

X-link: facebook/yoga#1389

This converts usages of YGMeasureMode to MeasureMode

Reviewed By: rozele

Differential Revision: D49271165

fbshipit-source-id: 03000f9c22096799ca1a61c1f9c168ce0933c34d
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 15, 2023
Summary:
X-link: facebook/react-native#39452

Pull Request resolved: facebook#1389

This converts usages of YGMeasureMode to MeasureMode

Reviewed By: rozele

Differential Revision: D49271165

fbshipit-source-id: a6ab7509a3f74bc6e1c42c0ce3376e1e7447f4dd
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49271165

facebook-github-bot pushed a commit to facebook/yoga that referenced this pull request Sep 15, 2023
Summary:
X-link: facebook/react-native#39452

Pull Request resolved: #1389

This converts usages of YGMeasureMode to MeasureMode

Reviewed By: rozele

Differential Revision: D49271165

fbshipit-source-id: 273c9ed0a61c3965e469548d29d37e4566c974dc
facebook-github-bot pushed a commit to facebook/litho that referenced this pull request Sep 15, 2023
Summary:
X-link: facebook/react-native#39452

X-link: facebook/yoga#1389

This converts usages of YGMeasureMode to MeasureMode

Reviewed By: rozele

Differential Revision: D49271165

fbshipit-source-id: 273c9ed0a61c3965e469548d29d37e4566c974dc
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 15, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 3b27c39.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants