From 886f4d049391993197b409a9038138a8208dd5fe Mon Sep 17 00:00:00 2001 From: Alexei Podtelezhnikov Date: Wed, 1 Apr 2015 22:57:43 -0400 Subject: [PATCH] [autofit] Finish the thought. * src/autofit/afhints.c (af_direction_compute): make sure the long arm is never negative so that its `FT_ABS' is not necessary. --- ChangeLog | 7 +++++++ src/autofit/afhints.c | 8 ++++---- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index ff20967ad..3200560e7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2015-03-21 Alexei Podtelezhnikov + + [autofit] Finish the thought. + + * src/autofit/afhints.c (af_direction_compute): make sure the long arm + is never negative so that its `FT_ABS' is not necessary. + 2015-04-01 Werner Lemberg [autofit] Call dumper functions for tracing. diff --git a/src/autofit/afhints.c b/src/autofit/afhints.c index 442cf6d0a..723e721d1 100644 --- a/src/autofit/afhints.c +++ b/src/autofit/afhints.c @@ -509,15 +509,15 @@ else { dir = AF_DIR_DOWN; - ll = dy; + ll = -dy; ss = dx; } } - /* return no direction if arm lengths differ too much */ + /* return no direction if arm lengths do not differ enough */ /* (value 14 is heuristic, corresponding to approx. 4.1 degrees) */ - ss *= 14; - if ( FT_ABS( ll ) <= FT_ABS( ss ) ) + /* the long arm is never negative */ + if ( ll <= 14 * FT_ABS( ss ) ) dir = AF_DIR_NONE; return dir;