Merge pull request #7723 from processing/fix/texttopoints-nan

Fix division by 0 in textToPoints

Author: davepagurek

src/type/p5.Font.js

@@ -1016,13 +1016,17 @@ function pathToPoints(cmds, options, font) {
     simplifyThreshold: 0
   });
 
-  const totalPoints = Math.ceil(path.getTotalLength() * opts.sampleFactor);
+  const totalPoints = Math.max(1, Math.ceil(path.getTotalLength() * opts.sampleFactor));
   let points = [];
 
   const mode = font._pInst.angleMode();
   const DEGREES = font._pInst.DEGREES;
   for (let i = 0; i < totalPoints; i++) {
-    const length = path.getTotalLength() * (i / (totalPoints - 1));
+    const length = path.getTotalLength() * (
+      totalPoints === 1
+        ? 0
+        : (i / (totalPoints - 1))
+    );
     points.push({
       ...path.getPointAtLength(length),
       get angle() {

test/unit/type/p5.Font.js

@@ -39,4 +39,15 @@ suite('p5.Font', function () {
     assert.property(bbox, 'w');
     assert.property(bbox, 'h');
   });
+
+  suite('textToPoints', () => {
+    test('contains no NaNs', async () => {
+      const pFont = await myp5.loadFont(fontFile);
+      const pts = pFont.textToPoints('hello, world!', 0, 0);
+      for (const pt of pts) {
+        expect(pt.x).not.toBeNaN();
+        expect(pt.y).not.toBeNaN();
+      }
+    });
+  });
 });