On 06/ 8/16 11:28 AM, Adam Jackson wrote:
gcc6 chirps an indentation warning here, but really this is bad code.
Effectively this would ignore en_US or en_UK names for the font, despite
that those are the English names the font is most likely to have.

Signed-off-by: Adam Jackson <[email protected]>
---
  src/FreeType/fttools.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/src/FreeType/fttools.c b/src/FreeType/fttools.c
index 7c0bd9b..0329fae 100644
--- a/src/FreeType/fttools.c
+++ b/src/FreeType/fttools.c
@@ -100,7 +100,6 @@ FTGetName(FT_Face face, int nid, int pid, int eid, 
FT_SfntName *name_return)
              case TT_PLATFORM_MICROSOFT:
                  if(name.language_id != TT_MS_LANGID_ENGLISH_UNITED_STATES &&
                     name.language_id != TT_MS_LANGID_ENGLISH_UNITED_KINGDOM)
-                    break;
                      continue;
                  break;
              default:


Reviewed-by: Alan Coopersmith <[email protected]>

--
        -Alan Coopersmith-              [email protected]
         Oracle Solaris Engineering - http://blogs.oracle.com/alanc
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to