Skip to content

Use canonical Objective-C super initializer guards - #58629

Closed
javache wants to merge 2 commits into
react:mainfrom
javache:export-D121046662
Closed

javache wants to merge 2 commits into
react:mainfrom
javache:export-D121046662

Conversation

@javache

@javache javache commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Summary:
Consolidate split superclass initializer assignments and validity checks into the canonical assignment-in-condition form. Configure readability-implicit-bool-conversion to allow pointer conditions so clang-tidy accepts that Objective-C idiom while retaining its other implicit-conversion coverage.

Changelog: [Internal]

Differential Revision: D121046662

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 21, 2026
@meta-codesync

meta-codesync Bot commented Sep 21, 2026

Copy link
Copy Markdown

@javache has exported this pull request. If you are a Meta employee, you can view the originating Diff in D121046662.

@meta-codesync meta-codesync Bot changed the title Normalize Objective-C super initializer guards Use canonical Objective-C super initializer guards Sep 21, 2026
Summary:
Restore idiomatic Objective-C fast enumeration now that React Native no longer enables the incompatible `cppcoreguidelines-init-variables` check.

Changelog: [Internal]

Differential Revision: D121040575
Summary:
Consolidate split superclass initializer assignments and validity checks into the canonical assignment-in-condition form. Configure `readability-implicit-bool-conversion` to allow pointer conditions so clang-tidy accepts that Objective-C idiom while retaining its other implicit-conversion coverage.

Changelog: [Internal]

Differential Revision: D121046662
@meta-codesync meta-codesync Bot closed this in 1774243 Sep 22, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 22, 2026
@meta-codesync

meta-codesync Bot commented Sep 22, 2026

Copy link
Copy Markdown

This pull request has been merged in 1774243.

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. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant