Skip to content

Commit

Permalink
Fix format specifier used in RCTUIManager
Browse files Browse the repository at this point in the history
Summary:
Here, `reactTag` is an `NSNumber *` which uses `%@` as the format specifier. Newer versions of clang can warn or error on this. This change prevents that from happening.

See also #15402

Local builds with clang 5.
Closes #15414

Differential Revision: D5583703

Pulled By: javache

fbshipit-source-id: a6d0d2246725cd096b1f3cd062ab5924740ab76c
  • Loading branch information
kastiglione authored and facebook-github-bot committed Aug 9, 2017
1 parent b06bfdd commit d81e549
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion React/Modules/RCTUIManager.m
Original file line number Diff line number Diff line change
Expand Up @@ -1421,7 +1421,7 @@ static void RCTMeasureLayout(RCTShadowView *view,
[self addUIBlock:^(__unused RCTUIManager *uiManager, NSDictionary<NSNumber *, UIView *> *viewRegistry) {
_jsResponder = viewRegistry[reactTag];
if (!_jsResponder) {
RCTLogError(@"Invalid view set to be the JS responder - tag %zd", reactTag);
RCTLogError(@"Invalid view set to be the JS responder - tag %@", reactTag);
}
}];
}
Expand Down

0 comments on commit d81e549

Please sign in to comment.