Skip to content

Fix unused parameter warning in propsConversions.h (#56500)#56500

Closed
christophpurrer wants to merge 1 commit into
react:mainfrom
christophpurrer:export-D101111638
Closed

Fix unused parameter warning in propsConversions.h (#56500)#56500
christophpurrer wants to merge 1 commit into
react:mainfrom
christophpurrer:export-D101111638

Conversation

@christophpurrer

@christophpurrer christophpurrer commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Summary:

Fixed clang-diagnostic-unused-parameter warning in React Native's propsConversions.h by commenting out the unused context parameter in the fromRawValue template function.

This change maintains API compatibility while resolving the compiler warning.

Changelog: [Internal]

Reviewed By: NickGerleman

Differential Revision: D101111638

@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 Apr 20, 2026
@meta-codesync

meta-codesync Bot commented Apr 20, 2026

Copy link
Copy Markdown

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

Summary:

Fixed clang-diagnostic-unused-parameter warning in React Native's propsConversions.h by commenting out the unused `context` parameter in the `fromRawValue` template function.

This change maintains API compatibility while resolving the compiler warning.


Changelog: [Internal]

Reviewed By: NickGerleman

Differential Revision: D101111638
@meta-codesync meta-codesync Bot changed the title Fix unused parameter warning in propsConversions.h Fix unused parameter warning in propsConversions.h (#56500) Apr 21, 2026
@meta-codesync meta-codesync Bot closed this in d655114 Apr 22, 2026
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @christophpurrer in d655114

When will my fix make it into a release? | How to file a pick request?

@meta-codesync

meta-codesync Bot commented Apr 22, 2026

Copy link
Copy Markdown

This pull request has been merged in d655114.

@facebook-github-tools facebook-github-tools Bot added the Merged This PR has been merged. label Apr 22, 2026
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. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants