ref: 886f4d049391993197b409a9038138a8208dd5fe
parent: a2fa657a03f17ef9cd79b4ad529f08cfe676aa1c
author: Alexei Podtelezhnikov <[email protected]>
date: Wed Apr 1 18:57:43 EDT 2015
[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.
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2015-03-21 Alexei Podtelezhnikov <[email protected]>
+
+ [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 <[email protected]>
[autofit] Call dumper functions for tracing.
--- 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;