Skip to content

Commit fd95f82

Browse files
generatedunixname89002005287564meta-codesync[bot]
authored andcommitted
Fix CQS signal readability-named-parameter in xplat/js/react-native-github/packages [A] (facebook#54026)
Summary: Pull Request resolved: facebook#54026 Reviewed By: javache Differential Revision: D83719624 fbshipit-source-id: 9e43d15dd4b44c9c7b3be711d62541135031d1f8
1 parent 47d1d2b commit fd95f82

File tree

12 files changed

+37
-25
lines changed

12 files changed

+37
-25
lines changed

packages/react-native/Libraries/Network/RCTHTTPRequestHandler.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ typedef NSURLSessionConfiguration * (^NSURLSessionConfigurationProvider)(void);
1313
* The block provided via this function will provide the NSURLSessionConfiguration for all HTTP requests made by the
1414
* app.
1515
*/
16-
RCT_EXTERN void RCTSetCustomNSURLSessionConfigurationProvider(NSURLSessionConfigurationProvider);
16+
RCT_EXTERN void RCTSetCustomNSURLSessionConfigurationProvider(NSURLSessionConfigurationProvider /*provider*/);
1717
/**
1818
* This is the default RCTURLRequestHandler implementation for HTTP requests.
1919
*/

packages/react-native/React/Base/RCTAssert.h

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,13 @@ RCT_EXTERN BOOL RCTIsMainQueue(void);
3737
do { \
3838
} while (false)
3939
#endif
40-
RCT_EXTERN void _RCTAssertFormat(const char *, const char *, int, const char *, NSString *, ...)
41-
NS_FORMAT_FUNCTION(5, 6);
40+
RCT_EXTERN void _RCTAssertFormat(
41+
const char * /*condition*/,
42+
const char * /*fileName*/,
43+
int /*lineNumber*/,
44+
const char * /*function*/,
45+
NSString * /*format*/,
46+
...) NS_FORMAT_FUNCTION(5, 6);
4247

4348
/**
4449
* Report a fatal condition when executing. These calls will _NOT_ be compiled out

packages/react-native/React/Base/RCTAssert.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
RCTFatalHandler RCTCurrentFatalHandler = nil;
2323
RCTFatalExceptionHandler RCTCurrentFatalExceptionHandler = nil;
2424

25-
NSException *_RCTNotImplementedException(SEL, Class);
25+
NSException *_RCTNotImplementedException(SEL /*cmd*/, Class /*cls*/);
2626
NSException *_RCTNotImplementedException(SEL cmd, Class cls)
2727
{
2828
NSString *msg = [NSString stringWithFormat:

packages/react-native/React/Base/RCTBridge.mm

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@
117117
}
118118

119119
static NSMutableArray<NSString *> *modulesLoadedWithOldArch;
120-
void addModuleLoadedWithOldArch(NSString *);
120+
void addModuleLoadedWithOldArch(NSString * /*moduleName*/);
121121
void addModuleLoadedWithOldArch(NSString *moduleName)
122122
{
123123
static dispatch_once_t onceToken;
@@ -138,7 +138,7 @@ void addModuleLoadedWithOldArch(NSString *moduleName)
138138
* prior to the first bridge initialization.
139139
* TODO: (T115656171) Refactor RCTRegisterModule out of Bridge.m since it doesn't use the Bridge.
140140
*/
141-
void RCTRegisterModule(Class);
141+
void RCTRegisterModule(Class /*moduleClass*/);
142142
void RCTRegisterModule(Class moduleClass)
143143
{
144144
if (RCTAreLegacyLogsEnabled() && ![getCoreModuleClasses() containsObject:[moduleClass description]]) {

packages/react-native/React/Base/RCTConvert.h

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -177,9 +177,14 @@ typedef NSArray UIColorArray __deprecated_msg("Use NSArray<UIColor *>");
177177
/**
178178
* Underlying implementations of RCT_XXX_CONVERTER macros. Ignore these.
179179
*/
180-
RCT_EXTERN NSNumber *RCTConvertEnumValue(const char *, NSDictionary *, NSNumber *, id);
181-
RCT_EXTERN NSNumber *RCTConvertMultiEnumValue(const char *, NSDictionary *, NSNumber *, id);
182-
RCT_EXTERN NSArray *RCTConvertArrayValue(SEL, id);
180+
RCT_EXTERN NSNumber *
181+
RCTConvertEnumValue(const char * /*typeName*/, NSDictionary * /*mapping*/, NSNumber * /*defaultValue*/, id /*json*/);
182+
RCT_EXTERN NSNumber *RCTConvertMultiEnumValue(
183+
const char * /*typeName*/,
184+
NSDictionary * /*mapping*/,
185+
NSNumber * /*defaultValue*/,
186+
id /*json*/);
187+
RCT_EXTERN NSArray *RCTConvertArrayValue(SEL /*type*/, id /*json*/);
183188

184189
/**
185190
* This macro is used for logging conversion errors. This is just used to

packages/react-native/React/Base/RCTLog.h

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,12 +75,12 @@ RCTFormatLog(NSDate *timestamp, RCTLogLevel level, NSString *fileName, NSNumber
7575
/**
7676
* A method to generate a string RCTLogLevel
7777
*/
78-
RCT_EXTERN NSString *RCTFormatLogLevel(RCTLogLevel);
78+
RCT_EXTERN NSString *RCTFormatLogLevel(RCTLogLevel /*level*/);
7979

8080
/**
8181
* A method to generate a string from a RCTLogSource
8282
*/
83-
RCT_EXTERN NSString *RCTFormatLogSource(RCTLogSource);
83+
RCT_EXTERN NSString *RCTFormatLogSource(RCTLogSource /*source*/);
8484

8585
/**
8686
* The default logging function used by RCTLogXX.
@@ -149,5 +149,7 @@ RCT_EXTERN void RCTLogSetBridgelessCallableJSModules(RCTCallableJSModules *calla
149149
} while (0)
150150
#endif
151151

152-
RCT_EXTERN void _RCTLogNativeInternal(RCTLogLevel, const char *, int, NSString *, ...) NS_FORMAT_FUNCTION(4, 5);
153-
RCT_EXTERN void _RCTLogJavaScriptInternal(RCTLogLevel, NSString *);
152+
RCT_EXTERN void
153+
_RCTLogNativeInternal(RCTLogLevel /*level*/, const char * /*fileName*/, int /*lineNumber*/, NSString * /*format*/, ...)
154+
NS_FORMAT_FUNCTION(4, 5);
155+
RCT_EXTERN void _RCTLogJavaScriptInternal(RCTLogLevel /*level*/, NSString * /*message*/);

packages/react-native/React/Base/RCTUtils.mm

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -706,7 +706,7 @@ BOOL RCTForceTouchAvailable(void)
706706
[data base64EncodedStringWithOptions:(NSDataBase64EncodingOptions)0]]];
707707
}
708708

709-
BOOL RCTIsGzippedData(NSData *__nullable); // exposed for unit testing purposes
709+
BOOL RCTIsGzippedData(NSData *__nullable /*data*/); // exposed for unit testing purposes
710710
BOOL RCTIsGzippedData(NSData *__nullable data)
711711
{
712712
UInt8 *bytes = (UInt8 *)data.bytes;

packages/react-native/React/CoreModules/RCTAppearance.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
#import <React/RCTEventEmitter.h>
1313

1414
RCT_EXTERN void RCTEnableAppearancePreference(BOOL enabled);
15-
RCT_EXTERN void RCTOverrideAppearancePreference(NSString *);
15+
RCT_EXTERN void RCTOverrideAppearancePreference(NSString * /*colorSchemeOverride*/);
1616
RCT_EXTERN void RCTUseKeyWindowForSystemStyle(BOOL useMainScreen);
1717
RCT_EXTERN NSString *RCTCurrentOverrideAppearancePreference(void);
1818
RCT_EXTERN NSString *RCTColorSchemePreference(UITraitCollection *traitCollection);

packages/react-native/React/CxxBridge/RCTMessageThread.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ class RCTMessageThread : public MessageQueueThread,
2121
public:
2222
RCTMessageThread(NSRunLoop* runLoop, RCTJavaScriptCompleteBlock errorBlock);
2323
~RCTMessageThread() override;
24-
void runOnQueue(std::function<void()>&&) override;
25-
void runOnQueueSync(std::function<void()>&&) override;
24+
void runOnQueue(std::function<void()>&& /*func*/) override;
25+
void runOnQueueSync(std::function<void()>&& /*func*/) override;
2626
void quitSynchronous() override;
2727
void setRunLoop(NSRunLoop* runLoop);
2828

packages/react-native/React/CxxBridge/RCTObjcExecutor.mm

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ void loadBundle(std::unique_ptr<const JSBigString> script, std::string sourceURL
9090
}];
9191
}
9292

93-
void setBundleRegistry(std::unique_ptr<RAMBundleRegistry>) override
93+
void setBundleRegistry(std::unique_ptr<RAMBundleRegistry> /*bundleRegistry*/) override
9494
{
9595
RCTAssert(NO, @"RAM bundles are not supported in RCTObjcExecutor");
9696
}

0 commit comments

Comments
 (0)