Re: [ft-devel] rendering differences between 2.4.2 and 2.4.4

[Алексей Подтележников]

Werner,

A more complete fix should include this patch. It removes a spurious shortcut.
Because it is spurious, I overlooked ONE_PIXEL / 4 there.
It also switches to a more natural do-while and removes unnecessary "return;"

My planned harmonization with the cubic is postponed indefinitely or
by a couple more
weeks. Please test.

Alexei

diff --git a/src/smooth/ftgrays.c b/src/smooth/ftgrays.c
index 648c75e..c4ede9f 100644
--- a/src/smooth/ftgrays.c
+++ b/src/smooth/ftgrays.c
@@ -890,12 +890,6 @@ typedef ptrdiff_t  FT_PtrDist;
     if ( dx < dy )
       dx = dy;

-    if ( dx <= ONE_PIXEL / 4 )
-    {
-      gray_render_line( RAS_VAR_ arc[0].x, arc[0].y );
-      return;
-    }
-
     level = 0;
     while ( dx > ONE_PIXEL / 6 )
     {
@@ -907,7 +901,7 @@ typedef ptrdiff_t  FT_PtrDist;
     levels[0] = level;
     top       = 0;

-    while ( top >= 0 )
+    do
     {
       level = levels[top];
       if ( level > 1 )
@@ -940,9 +934,8 @@ typedef ptrdiff_t  FT_PtrDist;
       gray_render_line( RAS_VAR_ arc[0].x, arc[0].y );
       top--;
       arc -= 2;
-    }

-    return;
+    } while ( top >= 0 );
   }