author Joshua Leung Sun, 9 Jun 2013 12:05:29 +0000 (12:05 +0000) committer Joshua Leung Sun, 9 Jun 2013 12:05:29 +0000 (12:05 +0000)
When you convert a grease pencil stroke to a polygon curve and look at the
vertices, the first and last vertex have weight = 0, but all others have a -NaN
value. This was caused by division by zero issues when minmax_weights ==
minmax_weights.

index 286fffb562abd061c3a5ac48467f0f4bab2bb143..9ee1f59fe211fe118ce1756d31be4c4954115f56 100644 (file)
@@ -1264,9 +1264,15 @@ static void gp_stroke_norm_curve_weights(Curve *cu, const float minmax_weights[2
{
Nurb *nu;
const float delta = minmax_weights;
{
Nurb *nu;
const float delta = minmax_weights;
-       const float fac = 1.0f / (minmax_weights - delta);
+       float fac;
int i;

int i;

+       /* when delta == minmax_weights == minmax_weights, we get div by zero [#35686] */
+       if (IS_EQ(delta, minmax_weights))
+               fac = 1.0f;
+       else
+               fac = 1.0f / (minmax_weights - delta);
+
for (nu = cu->nurb.first; nu; nu = nu->next) {
if (nu->bezt) {
BezTriple *bezt = nu->bezt;
for (nu = cu->nurb.first; nu; nu = nu->next) {
if (nu->bezt) {
BezTriple *bezt = nu->bezt;
@@ -1361,12 +1367,14 @@ static void gp_layer_to_curve(bContext *C, ReportList *reports, bGPdata *gpd, bG
}

/* If link_strokes, be sure first and last points have a zero weight/size! */
}

/* If link_strokes, be sure first and last points have a zero weight/size! */
gp_stroke_finalize_curve_endpoints(cu);
gp_stroke_finalize_curve_endpoints(cu);
-
+       }
+
/* Update curve's weights, if needed */
/* Update curve's weights, if needed */
-       if (norm_weights && ((minmax_weights > 0.0f) || (minmax_weights < 1.0f)))
+       if (norm_weights && ((minmax_weights > 0.0f) || (minmax_weights < 1.0f))) {
gp_stroke_norm_curve_weights(cu, minmax_weights);
gp_stroke_norm_curve_weights(cu, minmax_weights);
+       }

/* Create the path animation, if needed */
gp_stroke_path_animation(C, reports, cu, gtd);

/* Create the path animation, if needed */
gp_stroke_path_animation(C, reports, cu, gtd);