Move flatCoordsToPoints from util/util.js to pixi/helpers.js

We shouldn't be star importing from Pixi in our `util` module,
this is code that should be expected to be run headlessly in unit tests.

re: pixijs/pixijs#10170

modules/pixi/PixiFeaturePolygon.js

@@ -3,9 +3,8 @@ import { DashLine } from '@rapideditor/pixi-dashed-line';
 import { GlowFilter } from 'pixi-filters';
 import { /* geomRotatePoints,*/ vecEqual, vecLength /*, vecSubtract */ } from '@rapid-sdk/math';
 
-import { flatCoordsToPoints } from '../util/util.js';
 import { AbstractFeature } from './AbstractFeature.js';
-import { lineToPoly } from './helpers.js';
+import { flatCoordsToPoints, lineToPoly } from './helpers.js';
 
 const PARTIALFILLWIDTH = 32;
 

modules/pixi/helpers.js

@@ -332,6 +332,15 @@ if (isLimited && (span >= spacing * 100)) {
 }
 
 
+export function flatCoordsToPoints(coords) {
+  const points = new Array(coords.length / 2);
+  for (let i = 0; i < coords.length; i += 2) {
+    points[i / 2] = new PIXI.Point(coords[i], coords[i + 1]);
+  }
+  return points;
+}
+
+
 export function getDebugBBox(x, y, width, height, color, alpha, name) {
   const sprite = new PIXI.Sprite(PIXI.Texture.WHITE);
   sprite.eventMode = 'none';

modules/util/util.js

@@ -1,4 +1,3 @@
-import * as PIXI from 'pixi.js';
 import { Extent } from '@rapid-sdk/math';
 
 
@@ -25,15 +24,6 @@ export function utilTotalExtent(vals, graph) {
 }
 
 
-export function flatCoordsToPoints(coords) {
-  let points = [];
-  for (let i = 0; i < coords.length; i += 2){
-    points.push(new PIXI.Point(coords[i], coords[i + 1]));
-  }
-  return points;
-}
-
-
 // Adds or removes highlight styling for the specified entities
 export function utilHighlightEntities(entityIDs, highlighted, context) {
   const editor = context.systems.editor;