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

Define Flag operators for YGPrintOptions #37115

Closed
wants to merge 2 commits into from

Conversation

NickGerleman
Copy link
Contributor

Summary: This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Differential Revision: D45341504

@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 fb-exported labels Apr 27, 2023
@facebook-github-bot
Copy link
Contributor

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

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,623,485 -17
android hermes armeabi-v7a 7,936,344 +4
android hermes x86 9,110,433 -13
android hermes x86_64 8,965,253 +3
android jsc arm64-v8a 9,187,532 -6
android jsc armeabi-v7a 8,377,841 +7
android jsc x86 9,245,828 -16
android jsc x86_64 9,504,299 +9

Base commit: 33ebb5d
Branch: main

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Apr 27, 2023
Summary:
X-link: facebook/react-native#37115

Pull Request resolved: facebook#1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Differential Revision: D45341504

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

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

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Apr 27, 2023
Summary:
Pull Request resolved: facebook#37115

X-link: facebook/yoga#1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Differential Revision: D45341504

fbshipit-source-id: 96b582177407f62ef2ae85d6d90b512e7f30c8c4
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Apr 27, 2023
Summary:
X-link: facebook/react-native#37115

Pull Request resolved: facebook#1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Differential Revision: D45341504

fbshipit-source-id: 3bfec11d0d1a7b6a0f4b9e092f15a3c4d5bacd51
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Apr 27, 2023
Summary:
Pull Request resolved: facebook#37115

X-link: facebook/yoga#1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Differential Revision: D45341504

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

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

NickGerleman and others added 2 commits April 27, 2023 00:20
Differential Revision: D45254098

fbshipit-source-id: b057170421d6a748633142406a91a9ac7e10732a
Summary:
Pull Request resolved: facebook#37115

X-link: facebook/yoga#1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Reviewed By: rshest

Differential Revision: D45341504

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

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

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

Pull Request resolved: #1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Reviewed By: rshest

Differential Revision: D45341504

fbshipit-source-id: 0b80588f3e2e167d4c1c722c6d6608408dd617ba
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Apr 27, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in fe6f70b.

facebook-github-bot pushed a commit to facebook/litho that referenced this pull request Apr 27, 2023
Summary:
X-link: facebook/react-native#37115

X-link: facebook/yoga#1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Reviewed By: rshest

Differential Revision: D45341504

fbshipit-source-id: 0b80588f3e2e167d4c1c722c6d6608408dd617ba
jeongshin pushed a commit to jeongshin/react-native that referenced this pull request May 7, 2023
Summary:
Pull Request resolved: facebook#37115

X-link: facebook/yoga#1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Reviewed By: rshest

Differential Revision: D45341504

fbshipit-source-id: 0b80588f3e2e167d4c1c722c6d6608408dd617ba
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
Summary:
Pull Request resolved: facebook#37115

X-link: facebook/yoga#1267

This is an existing bitset enum in the API. Use the facility added in the last diff to add flag operators to it, to avoid the need for casting, and to make it clearer in the generated YGEnums.h that it is a bitset.

Reviewed By: rshest

Differential Revision: D45341504

fbshipit-source-id: 0b80588f3e2e167d4c1c722c6d6608408dd617ba
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