use method references to reduce recompositions

Signed-off-by: sds100 <developer.sds100@gmail.com>

Author: sds100

app/src/main/java/com/mapcode/map/InfoScreenPart.kt

@@ -3,10 +3,7 @@ package com.mapcode.map
 import androidx.compose.foundation.clickable
 import androidx.compose.foundation.layout.*
 import androidx.compose.material.ScaffoldState
-import androidx.compose.runtime.Composable
-import androidx.compose.runtime.collectAsState
-import androidx.compose.runtime.getValue
-import androidx.compose.runtime.rememberCoroutineScope
+import androidx.compose.runtime.*
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.unit.dp
@@ -147,11 +144,8 @@ fun InfoArea(
     val uiState by viewModel.uiState.collectAsState()
     val scope = rememberCoroutineScope()
     val copiedMessageStr = stringResource(R.string.copied_to_clipboard_snackbar_text)
-
-    InfoArea(
-        modifier,
-        uiState,
-        onMapcodeClick = {
+    val onMapcodeClick = remember {
+        {
             val copied = viewModel.copyMapcode()
             if (copied) {
                 scope.launch {
@@ -160,11 +154,17 @@ fun InfoArea(
                     scaffoldState.snackbarHostState.showSnackbar(copiedMessageStr)
                 }
             }
-        },
-        onAddressChange = { viewModel.queryAddress(it) },
-        onTerritoryClick = { viewModel.onTerritoryClick() },
-        onLatitudeChange = { viewModel.queryLatitude(it) },
-        onLongitudeChange = { viewModel.queryLongitude(it) },
+        }
+    }
+
+    InfoArea(
+        modifier,
+        uiState,
+        onMapcodeClick = onMapcodeClick,
+        onAddressChange = viewModel::queryAddress,
+        onTerritoryClick = viewModel::onTerritoryClick,
+        onLatitudeChange = viewModel::queryLatitude,
+        onLongitudeChange = viewModel::queryLongitude,
         isVerticalLayout = isVerticalLayout
     )
 }

app/src/main/java/com/mapcode/map/MapScreen.kt

@@ -197,7 +197,7 @@ fun MapControls(
 
     MapControls(
         modifier = modifier,
-        onSatelliteButtonClick = { viewModel.onSatelliteButtonClick() },
+        onSatelliteButtonClick = viewModel::onSatelliteButtonClick,
         isSatelliteModeEnabled = isSatelliteModeEnabled,
         onZoomInClick = {
             scope.launch {
@@ -222,8 +222,8 @@ fun MapControls(
                 locationPermissionsState.launchMultiplePermissionRequest()
             }
         },
-        onDirectionsClick = { viewModel.onDirectionsClick() },
-        onShareMapcodeClick = { viewModel.shareMapcode() },
+        onDirectionsClick = viewModel::onDirectionsClick,
+        onShareMapcodeClick = viewModel::shareMapcode,
         onAboutClick = { showAboutDialog = true }
     )
 }
@@ -367,7 +367,7 @@ fun MapWithCrossHairs(
         if (renderGoogleMaps) {
             Map(
                 properties = viewModel.mapProperties,
-                onCameraMoved = { lat, long, zoom -> viewModel.onCameraMoved(lat, long, zoom) },
+                onCameraMoved = viewModel::onCameraMoved,
                 cameraPositionState = viewModel.cameraPositionState
             )
         }