From e3e3d13fa523ce87202b9dc8992cb01263c1056d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ju=CC=88rg=20Lehni?= Date: Fri, 2 Jan 2015 16:33:15 +0100 Subject: [PATCH] Add comment regarding issue #565 --- src/path/Curve.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/path/Curve.js b/src/path/Curve.js index 1a038db0..76b637d7 100644 --- a/src/path/Curve.js +++ b/src/path/Curve.js @@ -1056,6 +1056,11 @@ new function() { // Scope for methods that require numerical integration tMin, tMax, uMin, uMax, oldTDiff, reverse, recursion) { /*#*/ if (__options.fatlineClipping) { // Avoid deeper recursion. + // NOTE: @iconexperience determined that more than 20 recursions are + // needed sometimes, depending on the tDiff threshold values further + // below when determining which curve converges the least. He also + // recommended a threshold of 0.5 instead of the initial 0.8 + // See: https://github.com/paperjs/paper.js/issues/565 if (recursion > 32) return; // Let P be the first curve and Q be the second