maplibre / flutter-maplibre-gl

Customizable, performant and vendor-free vector and raster maps, flutter wrapper for maplibre-native and maplibre-gl-js (fork of flutter-mapbox-gl/maps)
https://pub.dev/packages/maplibre_gl
Other
226 stars 125 forks source link

[FEATURE] Migrate from manual method_channel to Pigeon #465

Open T-moz opened 5 months ago

T-moz commented 5 months ago

Feature Description

Currently, communication between dart and the native side is done by using method channel int the MapLibreMethodChannel class. This works, but it's hard to maintain, it's not type safe, and the usage of jsonEncode in the main isolate causes performances issues.

Describe alternatives you've considered

Replacing the method channel with Pigeon.

Additional context

No response

kuhnroyal commented 5 months ago

Are you willing to contribute a PR?

josxha commented 5 months ago

Great suggestion @T-moz! I'd love to see a deacrese in code that is not type safe and Pigeon seems like a great choice.