Skip to content

feat!: rewrite android maps utils#1658

Open
kikoso wants to merge 84 commits into
mainfrom
feat/rewrite-android-maps-utils
Open

feat!: rewrite android maps utils#1658
kikoso wants to merge 84 commits into
mainfrom
feat/rewrite-android-maps-utils

fix(geojson): prevent StackOverflowError when parsing deeply nested G…

936a1f1
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / Android Lint succeeded Jun 24, 2026 in 3s

37 new alerts

New alerts in code changed by this pull request

  • 37 warnings

Alerts not introduced by this pull request might have been detected because the code changes were too large.

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 24 in demo/build.gradle.kts

See this annotation in the file changed.

Code scanning / Android Lint

Newer Library Versions Available Warning

A newer version of org.jetbrains.kotlin.plugin.serialization than 2.2.21 is available: 2.4.0

Check warning on line 76 in demo/build.gradle.kts

See this annotation in the file changed.

Code scanning / Android Lint

Target SDK attribute is not targeting latest version Warning

Not targeting the latest versions of Android; compatibility modes apply. Consider testing and updating this version. Consult the android.os.Build.VERSION_CODES javadoc for details.

Check warning on line 1 in demo/src/main/AndroidManifest.xml

See this annotation in the file changed.

Code scanning / Android Lint

Using RTL attributes without enabling RTL support Warning

The project references RTL attributes, but does not explicitly enable or disable RTL support with android:supportsRtl in the manifest

Check warning on line 27 in demo/src/main/AndroidManifest.xml

See this annotation in the file changed.

Code scanning / Android Lint

Affected by scoped storage Warning

WRITE_EXTERNAL_STORAGE is deprecated (and is not granted) when targeting Android 13+. If you need to write to shared storage, use the MediaStore.createWriteRequest intent.

Check warning on line 38 in demo/src/main/AndroidManifest.xml

See this annotation in the file changed.

Code scanning / Android Lint

Missing data extraction rules Warning

The attribute android:allowBackup is deprecated from Android 12 and higher and may be removed in future versions. Consider adding the attribute android:dataExtractionRules specifying an @xml resource which configures cloud backups and device transfers on Android 12 and higher.

Check warning on line 35 in demo/src/main/java/com/google/maps/android/utils/demo/AnimationUtilDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 148 in demo/src/main/java/com/google/maps/android/utils/demo/ClusterAlgorithmsDemoActivity.kt

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 59 in demo/src/main/java/com/google/maps/android/utils/demo/ClusteringDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Layout Inflation without a Parent Warning

Avoid passing null as the view root (needed to resolve layout parameters on the inflated layout's root element)

Check warning on line 152 in demo/src/main/java/com/google/maps/android/utils/demo/ClusteringDiffDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 153 in demo/src/main/java/com/google/maps/android/utils/demo/ClusteringDiffDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 185 in demo/src/main/java/com/google/maps/android/utils/demo/CustomAdvancedMarkerClusteringDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 186 in demo/src/main/java/com/google/maps/android/utils/demo/CustomAdvancedMarkerClusteringDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 65 in demo/src/main/java/com/google/maps/android/utils/demo/CustomMarkerClusteringDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Layout Inflation without a Parent Warning

Avoid passing null as the view root (needed to resolve layout parameters on the inflated layout's root element)

Check warning on line 134 in demo/src/main/java/com/google/maps/android/utils/demo/CustomMarkerClusteringDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Should not call Context.getDrawable or Resources.getDrawable directly Warning

Use ResourcesCompat.getDrawable()

Check warning on line 207 in demo/src/main/java/com/google/maps/android/utils/demo/CustomMarkerClusteringDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 208 in demo/src/main/java/com/google/maps/android/utils/demo/CustomMarkerClusteringDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 51 in demo/src/main/java/com/google/maps/android/utils/demo/DistanceDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY Warning

Using this method may override behaviors set by the Maps SDK for Android Utility Library. If you are not using clustering, GeoJson, or KML, you can safely suppress this warning, otherwise, refer to the utility library's migration guide: https://bit.ly/3kTpQmY

Check warning on line 86 in demo/src/main/java/com/google/maps/android/utils/demo/DistanceDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Implied default locale in case conversion Warning

Implicitly using the default locale is a common source of bugs: Use String.format(Locale, ...) instead

Check warning on line 113 in demo/src/main/java/com/google/maps/android/utils/demo/GeoJsonDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Static Field Leaks Warning

This AsyncTask class should be static or leaks might occur (com.google.maps.android.utils.demo.GeoJsonDemoActivity.DownloadGeoJsonFile)

Check warning on line 324 in demo/src/main/java/com/google/maps/android/utils/demo/HeatmapsPlacesDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Static Field Leaks Warning

This AsyncTask class should be static or leaks might occur (com.google.maps.android.utils.demo.HeatmapsPlacesDemoActivity.MakeOverlayTask)

Check warning on line 143 in demo/src/main/java/com/google/maps/android/utils/demo/KmlDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Static Field Leaks Warning

This AsyncTask class should be static or leaks might occur (com.google.maps.android.utils.demo.KmlDemoActivity.LoadLocalKmlFile)

Check warning on line 168 in demo/src/main/java/com/google/maps/android/utils/demo/KmlDemoActivity.java

See this annotation in the file changed.

Code scanning / Android Lint

Static Field Leaks Warning

This AsyncTask class should be static or leaks might occur (com.google.maps.android.utils.demo.KmlDemoActivity.DownloadKmlFile)

Check warning on line 21 in demo/src/main/res/drawable/bg_circle_white.xml

See this annotation in the file changed.

Code scanning / Android Lint

Unused resources Warning

The resource R.drawable.bg_circle_white appears to be unused

Check warning on line 17 in demo/src/main/res/layout/custom_info_window.xml

See this annotation in the file changed.

Code scanning / Android Lint

Overdraw: Painting regions more than once Warning

Possible overdraw: Root element paints background @android:color/white with a theme that also paints a background (inferred theme is @style/AppTheme)

Check warning on line 28 in demo/src/main/res/layout/heatmaps_demo.xml

See this annotation in the file changed.

Code scanning / Android Lint

Button should be borderless Warning

Buttons in button bars should be borderless; use style="?android:attr/buttonBarButtonStyle" (and ?android:attr/buttonBarStyle on the parent)