jens/flipper
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Back out "Use component.className instead of NSStringFromClass"

Browse Source 
Summary:
Original diff broke OSS flipper. Need a new OSS release of CK first

Original commit changeset: 3e7564b25610

Reviewed By: kfirapps

Differential Revision: D20304743

fbshipit-source-id: 08a56359a42bc56b576d212e7cf642197a064076
This commit is contained in:
Hugo Cuvillier
2020-03-06 07:56:28 -08:00
committed by Facebook Github Bot
parent 5a4ac4c32d
commit 987822074b
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
iOS/Plugins/FlipperKitLayoutPlugin/FlipperKitLayoutComponentKitSupport/CKComponent+Sonar.mm
View File

@@ -109,10 +109,10 @@ static CK::StaticMutex _mutex = CK_MUTEX_INITIALIZER;
auto const canBeReusedCounter = self.flipper_canBeReusedCounter; auto const canBeReusedCounter = self.flipper_canBeReusedCounter;
if (canBeReusedCounter > 0) { if (canBeReusedCounter > 0) {
return [NSString stringWithFormat:@"%@ (Can be reused x%lu)", return [NSString stringWithFormat:@"%@ (Can be reused x%lu)",
self.className, NSStringFromClass([self class]),
(unsigned long)canBeReusedCounter]; (unsigned long)canBeReusedCounter];
} }
return self.className; return NSStringFromClass([self class]);
} }
- (NSString*)sonar_getDecoration { - (NSString*)sonar_getDecoration {

2
iOS/Plugins/FlipperKitLayoutPlugin/FlipperKitLayoutComponentKitSupport/SKComponentLayoutWrapper.mm
View File

@@ -98,7 +98,7 @@ static CKFlexboxComponentChild findFlexboxLayoutParams(
_size = layout.size; _size = layout.size;
_position = position; _position = position;
_identifier = [parentKey stringByAppendingString:layout.component _identifier = [parentKey stringByAppendingString:layout.component
? layout.component.className ? NSStringFromClass([layout.component class])
: @"(null)"]; : @"(null)"];
if (_component && reuseWrapper) { if (_component && reuseWrapper) {