Skip to content

Conversation

@yannickl
Copy link
Contributor

@yannickl yannickl commented Jan 6, 2026

Summary:
In some configurations the errors might be stripped out and leading toward those kind of issues:

stderr:

xplat/js/react-native-github/packages/react-native/React/Modules/RCTUIManager.mm:1251:28: error: unused exception parameter 'exception' [-Werror,-Wunused-exception-parameter]

 1251 |     } catch (NSException \*exception) {

      |                            ^\~\~\~\~\~\~\~\~

1 error generated.

So marking the param as potentially unused.

Reviewed By: lodhaayush

Differential Revision: D90192993

Summary:
In some configurations the errors might be stripped out and leading toward those kind of issues:

```
stderr:

xplat/js/react-native-github/packages/react-native/React/Modules/RCTUIManager.mm:1251:28: error: unused exception parameter 'exception' [-Werror,-Wunused-exception-parameter]

 1251 |     } catch (NSException \*exception) {

      |                            ^\~\~\~\~\~\~\~\~

1 error generated.
```

So marking the param as potentially unused.

Reviewed By: lodhaayush

Differential Revision: D90192993
@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 Jan 6, 2026
@meta-codesync
Copy link

meta-codesync bot commented Jan 6, 2026

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

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 meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants