From c618221a388b2e5c842c9c3952157e68e5769730 Mon Sep 17 00:00:00 2001
From: mar-v-in <github@rvin.mooo.com>
Date: Thu, 29 Oct 2015 12:04:37 +0100
Subject: [PATCH] Maps: Properly handle invisible map items and impossible
 lines

Fixes #49
---
 .../src/main/java/org/microg/gms/maps/BackendMap.java       | 6 +++++-
 .../main/java/org/microg/gms/maps/markup/PolylineImpl.java  | 4 ++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/play-services-core/src/main/java/org/microg/gms/maps/BackendMap.java b/play-services-core/src/main/java/org/microg/gms/maps/BackendMap.java
index 1bbbd5cc..5bf81405 100644
--- a/play-services-core/src/main/java/org/microg/gms/maps/BackendMap.java
+++ b/play-services-core/src/main/java/org/microg/gms/maps/BackendMap.java
@@ -38,6 +38,7 @@ import org.oscim.layers.marker.MarkerSymbol;
 import org.oscim.layers.tile.buildings.BuildingLayer;
 import org.oscim.layers.tile.vector.VectorTileLayer;
 import org.oscim.layers.tile.vector.labeling.LabelLayer;
+import org.oscim.layers.vector.geometries.Drawable;
 import org.oscim.map.Layers;
 import org.oscim.map.Map;
 import org.oscim.map.Viewport;
@@ -171,7 +172,10 @@ public class BackendMap implements ItemizedLayer.OnItemGestureListener<MarkerIte
     private synchronized void updateDrawableLayer() {
         drawables.clear();
         for (DrawableMarkup markup : drawableMarkups) {
-            drawables.add(markup.getDrawable(mapView.map()));
+            Drawable drawable = markup.getDrawable(mapView.map());
+            if (drawable != null) {
+                drawables.add(drawable);
+            }
         }
     }
 
diff --git a/play-services-core/src/main/java/org/microg/gms/maps/markup/PolylineImpl.java b/play-services-core/src/main/java/org/microg/gms/maps/markup/PolylineImpl.java
index 1f601584..9db016c0 100644
--- a/play-services-core/src/main/java/org/microg/gms/maps/markup/PolylineImpl.java
+++ b/play-services-core/src/main/java/org/microg/gms/maps/markup/PolylineImpl.java
@@ -148,6 +148,10 @@ public class PolylineImpl extends IPolylineDelegate.Stub implements DrawableMark
     @Override
     public Drawable getDrawable(Map map) {
         if (!isVisible() || removed) return null;
+        if (options.getPoints().size() < 2) {
+            // You hardly draw a line with less than two points
+            return null;
+        }
         List<GeoPoint> points = new ArrayList<GeoPoint>();
         for (LatLng point : options.getPoints()) {
             points.add(GmsMapsTypeHelper.fromLatLng(point));