fix: improve code quality

jokerttu · jokerttu · commit ffcc10e0dc45 · 2025-11-07T17:00:53.000+02:00
diff --git a/packages/google_maps_flutter/google_maps_flutter_android/android/src/main/java/io/flutter/plugins/googlemaps/ClusterManagersController.java b/packages/google_maps_flutter/google_maps_flutter_android/android/src/main/java/io/flutter/plugins/googlemaps/ClusterManagersController.java
@@ -111,7 +111,6 @@ void addClusterManager(String clusterManagerId) {
    * advanced markers and MarkerClusterRenderer is used for default markers.
    */
   private void initializeRenderer(ClusterManager<MarkerBuilder> clusterManager) {
-    final ClusterRenderer<MarkerBuilder> renderer = clusterManager.getRenderer();
     final ClusterRenderer<MarkerBuilder> clusterRenderer =
         markerType == PlatformMarkerType.ADVANCED_MARKER
             ? new AdvancedMarkerClusterRenderer<>(context, googleMap, clusterManager, this)
diff --git a/packages/google_maps_flutter/google_maps_flutter_android/android/src/main/java/io/flutter/plugins/googlemaps/Convert.java b/packages/google_maps_flutter/google_maps_flutter_android/android/src/main/java/io/flutter/plugins/googlemaps/Convert.java
@@ -733,8 +733,9 @@ static int collisionBehaviorFromPigeon(
         return AdvancedMarkerOptions.CollisionBehavior.OPTIONAL_AND_HIDES_LOWER_PRIORITY;
       case REQUIRED_AND_HIDES_OPTIONAL:
         return AdvancedMarkerOptions.CollisionBehavior.REQUIRED_AND_HIDES_OPTIONAL;
+      default:
+        return AdvancedMarkerOptions.CollisionBehavior.REQUIRED;
     }
-    return AdvancedMarkerOptions.CollisionBehavior.REQUIRED;
   }
 
   static String interpretPolylineOptions(

Original file line number	Diff line number	Diff line change
`@@ -733,8 +733,9 @@ static int collisionBehaviorFromPigeon(`
`733`	`733`	`return AdvancedMarkerOptions.CollisionBehavior.OPTIONAL_AND_HIDES_LOWER_PRIORITY;`
`734`	`734`	`case REQUIRED_AND_HIDES_OPTIONAL:`
`735`	`735`	`return AdvancedMarkerOptions.CollisionBehavior.REQUIRED_AND_HIDES_OPTIONAL;`
	`736`	`+ default:`
	`737`	`+ return AdvancedMarkerOptions.CollisionBehavior.REQUIRED;`
`736`	`738`	`}`
`737`		`- return AdvancedMarkerOptions.CollisionBehavior.REQUIRED;`
`738`	`739`	`}`
`739`	`740`
`740`	`741`	`static String interpretPolylineOptions(`