Optimize PaddingValues copy

Avoid nullable parameters in this function and we can let local direction as the default param.
android · Mar 28, 2024 · 27fe258 · 27fe258
1 parent e2d0daa
commit 27fe258
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/app/src/main/java/com/google/android/samples/socialite/ui/chat/ChatScreen.kt b/app/src/main/java/com/google/android/samples/socialite/ui/chat/ChatScreen.kt
@@ -229,17 +229,18 @@ private fun ChatContent(
     }
 }
 
+@Composable
 private fun PaddingValues.copy(
-    layoutDirection: LayoutDirection,
-    start: Dp? = null,
-    top: Dp? = null,
-    end: Dp? = null,
-    bottom: Dp? = null,
+    layoutDirection: LayoutDirection = LocalLayoutDirection.current,
+    start: Dp = calculateStartPadding(layoutDirection),
+    top: Dp = calculateTopPadding(),
+    end: Dp = calculateEndPadding(layoutDirection),
+    bottom: Dp = calculateBottomPadding(),
 ) = PaddingValues(
-    start = start ?: calculateStartPadding(layoutDirection),
-    top = top ?: calculateTopPadding(),
-    end = end ?: calculateEndPadding(layoutDirection),
-    bottom = bottom ?: calculateBottomPadding(),
+    start = start,
+    top = top,
+    end = end,
+    bottom = bottom,
 )
 
 @OptIn(ExperimentalMaterial3Api::class)