Henri Verbeet : usp10: Return early if the number of positioning operations is 0 in GPOS_apply_ChainContextPos ().

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 038976856805c9bec00291e6784310f24e9168f5
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=038976856805c9bec00291e678...

Author: Henri Verbeet 
Date:   Wed Apr  5 10:21:29 2017 +0200

usp10: Return early if the number of positioning operations is 0 in GPOS_apply_ChainContextPos().

This preserves the current logic, but I wonder if we're not supposed to always
return "input_count". Returning 1 seems questionable if we matched more input
glyphs than that, and so does including the lookahead count.

Signed-off-by: Henri Verbeet 
Signed-off-by: Aric Stewart 
Signed-off-by: Alexandre Julliard

---

dlls/usp10/opentype.c | 23 +++++++++++------------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/dlls/usp10/opentype.c b/dlls/usp10/opentype.c
index 9c1d596..0d98253 100644
--- a/dlls/usp10/opentype.c
+++ b/dlls/usp10/opentype.c
@@ -2226,7 +2226,7 @@ static unsigned int GPOS_apply_ChainContextPos(const ScriptCache *script_cache,
         }
         else if (GET_BE_WORD(backtrack->PosFormat) == 3)
         {
-            WORD backtrack_count, input_count, lookahead_count;
+            WORD backtrack_count, input_count, lookahead_count, positioning_count;
             int k;
             const GPOS_ChainContextPosFormat3_2 *input;
             const GPOS_ChainContextPosFormat3_3 *lookahead;
@@ -2286,20 +2286,19 @@ static unsigned int GPOS_apply_ChainContextPos(const ScriptCache *script_cache,
                 continue;
             TRACE("Matched LookAhead\n");

-            if (GET_BE_WORD(positioning->PosCount))
+            if (!(positioning_count = GET_BE_WORD(positioning->PosCount)))
+                return 1;
+
+            for (k = 0; k < positioning_count; ++k)
             {
-                for (k = 0; k < GET_BE_WORD(positioning->PosCount); ++k)
-                {
-                    int lookupIndex = GET_BE_WORD(positioning->PosLookupRecord[k].LookupListIndex);
-                    int SequenceIndex = GET_BE_WORD(positioning->PosLookupRecord[k].SequenceIndex) * write_dir;
+                WORD lookup_index = GET_BE_WORD(positioning->PosLookupRecord[k].LookupListIndex);
+                WORD sequence_index = GET_BE_WORD(positioning->PosLookupRecord[k].SequenceIndex) * write_dir;

-                    TRACE("Position: %i -> %i %i\n",k, SequenceIndex, lookupIndex);
-                    GPOS_apply_lookup(script_cache, otm, logfont, analysis, advance, lookup, lookupIndex,
-                            glyphs, glyph_index + SequenceIndex, glyph_count, goffset);
-                }
-                return input_count + lookahead_count;
+                TRACE("Position: %u -> %u %u.\n", k, sequence_index, lookup_index);
+                GPOS_apply_lookup(script_cache, otm, logfont, analysis, advance, lookup, lookup_index,
+                        glyphs, glyph_index + sequence_index, glyph_count, goffset);
             }
-            else return 1;
+            return input_count + lookahead_count;
         }
         else
             FIXME("Unhandled Chaining Contextual Positioning Format %#x.\n", GET_BE_WORD(backtrack->PosFormat));