populate paint arrays once per line feature (#9452)

instead of once per linestring in a multiline
mapbox · Mar 24, 2020 · eeab959 · eeab959
1 parent a2afce6
commit eeab959
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/data/bucket/line_bucket.js b/src/data/bucket/line_bucket.js
@@ -219,11 +219,13 @@ class LineBucket implements Bucket {
         const roundLimit = layout.get('line-round-limit');
 
         for (const line of geometry) {
-            this.addLine(line, feature, join, cap, miterLimit, roundLimit, index, canonical, imagePositions);
+            this.addLine(line, feature, join, cap, miterLimit, roundLimit);
         }
+
+        this.programConfigurations.populatePaintArrays(this.layoutVertexArray.length, feature, index, imagePositions, canonical);
     }
 
-    addLine(vertices: Array<Point>, feature: BucketFeature, join: string, cap: string, miterLimit: number, roundLimit: number, index: number, canonical: CanonicalTileID, imagePositions: {[_: string]: ImagePosition}) {
+    addLine(vertices: Array<Point>, feature: BucketFeature, join: string, cap: string, miterLimit: number, roundLimit: number) {
         this.distance = 0;
         this.scaledDistance = 0;
         this.totalDistance = 0;
@@ -468,8 +470,6 @@ class LineBucket implements Bucket {
                 }
             }
         }
-
-        this.programConfigurations.populatePaintArrays(this.layoutVertexArray.length, feature, index, imagePositions, canonical);
     }
 
     /**