feat: align unnecessary compiler conditionals (#51)

callstack · Mar 6, 2024 · 57354a3 · 57354a3
1 parent 7c2c397
commit 57354a3
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 8 deletions.
diff --git a/packages/react-native/React/Base/RCTUtils.m b/packages/react-native/React/Base/RCTUtils.m
@@ -315,23 +315,17 @@ static void RCTUnsafeExecuteOnMainQueueOnceSync(dispatch_once_t *onceToken, disp
 
 void RCTComputeScreenScale(void)
 {
-#if !TARGET_OS_VISION
   dispatch_once(&onceTokenScreenScale, ^{
     screenScale = [UITraitCollection currentTraitCollection].displayScale;
   });
-#endif
 }
 
 CGFloat RCTScreenScale(void)
 {
-#if !TARGET_OS_VISION
   RCTUnsafeExecuteOnMainQueueOnceSync(&onceTokenScreenScale, ^{
     screenScale = [UITraitCollection currentTraitCollection].displayScale;
   });
   return screenScale;
-#endif
-
-  return 2;
 }
 
 CGFloat RCTFontSizeMultiplier(void)

diff --git a/...act-native/React/Fabric/Mounting/ComponentViews/Modal/RCTFabricModalHostViewController.mm b/...act-native/React/Fabric/Mounting/ComponentViews/Modal/RCTFabricModalHostViewController.mm
@@ -42,6 +42,7 @@ - (void)loadView
   [_touchHandler attachToView:self.view];
 }
 
+#if !TARGET_OS_VISION
 - (UIStatusBarStyle)preferredStatusBarStyle
 {
   return [RCTUIStatusBarManager() statusBarStyle];
@@ -53,6 +54,7 @@ - (void)viewDidDisappear:(BOOL)animated
   _lastViewBounds = CGRectZero;
 }
 
+#if !TARGET_OS_VISION
 - (BOOL)prefersStatusBarHidden
 {
   return [RCTUIStatusBarManager() isStatusBarHidden];

diff --git a/packages/react-native/React/UIUtils/RCTUIUtils.m b/packages/react-native/React/UIUtils/RCTUIUtils.m
@@ -22,7 +22,7 @@ RCTDimensions RCTGetDimensions(CGFloat fontScale)
   CGSize windowSize = mainWindow ? mainWindow.bounds.size : screenSize;
   CGFloat scale;
 #if TARGET_OS_VISION
-    scale = 2;
+    scale = [UITraitCollection currentTraitCollection].displayScale;
 #else
     scale = mainScreen.scale;
 #endif

diff --git a/packages/react-native/React/Views/RCTModalHostView.m b/packages/react-native/React/Views/RCTModalHostView.m
@@ -78,7 +78,7 @@ - (void)notifyForOrientationChange
 #if !TARGET_OS_VISION
   UIInterfaceOrientation currentOrientation = [RCTSharedApplication() statusBarOrientation];
 #else
-    UIInterfaceOrientation currentOrientation = UIDeviceOrientationUnknown;
+    UIInterfaceOrientation currentOrientation = UIInterfaceOrientationUnknown;
 #endif
   if (currentOrientation == _lastKnownOrientation) {
     return;