Skip to content

Commit dd6805d

Browse files
r-barnesmeta-codesync[bot]
authored andcommitted
Remove unused exception parameter from js/react-native-github/packages/react-native/React/CxxModule/RCTCxxUtils.mm (#54539)
Summary: Pull Request resolved: #54539 `-Wunused-exception-parameter` has identified an unused exception parameter. This diff removes it. This: ``` try { ... } catch (exception& e) { // no use of e } ``` should instead be written as ``` } catch (exception&) { ``` If the code compiles, this is safe to land. Differential Revision: D85813832 fbshipit-source-id: 4edbaaf59a06db619e85d272e77b55081623fed5
1 parent 1859245 commit dd6805d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

packages/react-native/ReactCommon/react/nativemodule/core/platform/ios/ReactCommon/RCTTurboModuleManager.mm

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -694,7 +694,7 @@ - (BOOL)_shouldCreateObjCModule:(Class)moduleClass
694694
} else if (_bridgeProxy) {
695695
[(id)module setValue:_bridgeProxy forKey:@"bridge"];
696696
}
697-
} @catch (NSException *exception) {
697+
} @catch (NSException *) {
698698
RCTLogError(
699699
@"%@ has no setter or ivar for its bridge, which is not "
700700
"permitted. You must either @synthesize the bridge property, "
@@ -742,7 +742,7 @@ - (BOOL)_shouldCreateObjCModule:(Class)moduleClass
742742

743743
@try {
744744
[(id)module setValue:methodQueue forKey:@"methodQueue"];
745-
} @catch (NSException *exception) {
745+
} @catch (NSException *) {
746746
RCTLogError(
747747
@"%@ has no setter or ivar for its methodQueue, which is not "
748748
"permitted. You must either @synthesize the methodQueue property, "

0 commit comments

Comments
 (0)