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

Simplify YGConfigSetPointScaleFactor #1451

Closed
wants to merge 2 commits into from

Conversation

NickGerleman
Copy link
Contributor

Summary: The special meaning of 0.0 is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Differential Revision: D51014264

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 10, 2023
Summary:

X-link: facebook/react-native#41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 10, 2023
Summary:

X-link: facebook/react-native#41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 10, 2023
Summary:
X-link: facebook/yoga#1451


The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 10, 2023
Summary:
X-link: facebook/yoga#1451


The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 10, 2023
Summary:

X-link: facebook/react-native#41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 10, 2023
Summary:

X-link: facebook/react-native#41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 10, 2023
Summary:
X-link: facebook/yoga#1451


The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 10, 2023
Summary:

X-link: facebook/react-native#41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 10, 2023
Summary:
X-link: facebook/yoga#1451


The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
Summary:
X-link: facebook/react-native#41317


This aims to clean up the public Yoga C API, by:
1. Documenting public YGNode, YGValue, YGConfig APIs
2. Splitting APIs for specific objects into different header files (because Yoga.h was big enough without documentation)
3. Reordering headers and definitions for consistent grouping

Changelog: [Internal]

Reviewed By: joevilches

Differential Revision: D50963424
Summary:

X-link: facebook/react-native#41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 11, 2023
Summary:
X-link: facebook/yoga#1451


The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 1b1df3a.

facebook-github-bot pushed a commit to facebook/litho that referenced this pull request Nov 11, 2023
Summary:
X-link: facebook/yoga#1451

X-link: facebook/react-native#41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264

fbshipit-source-id: 60f4a2df039f74089d5c7fabd4b7d8ac6234ba72
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Nov 11, 2023
Summary:
X-link: facebook/yoga#1451

Pull Request resolved: #41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264

fbshipit-source-id: 60f4a2df039f74089d5c7fabd4b7d8ac6234ba72
Othinn pushed a commit to Othinn/react-native that referenced this pull request Jan 9, 2024
Summary:
X-link: facebook/yoga#1451

Pull Request resolved: facebook#41327

The special meaning of `0.0` is now explained in the function header, and we aren't doing any sort of insensitive compare here, so the code after should be equivalent and a bit simpler.

Reviewed By: yungsters

Differential Revision: D51014264

fbshipit-source-id: 60f4a2df039f74089d5c7fabd4b7d8ac6234ba72
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants