Byeong-Sik Jeon : gdi32: Fix wrong break statement use.

Alexandre Julliard julliard at wine.codeweavers.com
Wed Sep 13 06:07:03 CDT 2006


Module: wine
Branch: master
Commit: 09228971645e4eafc3dd3bcec317494f663c2fea
URL:    http://source.winehq.org/git/?p=wine.git;a=commit;h=09228971645e4eafc3dd3bcec317494f663c2fea

Author: Byeong-Sik Jeon <wjsqudtlr at gmail.com>
Date:   Wed Sep 13 06:41:43 2006 +0900

gdi32: Fix wrong break statement use.

---

 dlls/gdi/freetype.c |   11 ++++++-----
 1 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/dlls/gdi/freetype.c b/dlls/gdi/freetype.c
index a664db9..ee51519 100644
--- a/dlls/gdi/freetype.c
+++ b/dlls/gdi/freetype.c
@@ -451,7 +451,6 @@ static Face *find_face_from_filename(con
     const char *file;
     DWORD len = WideCharToMultiByte(CP_UNIXCP, 0, file_name, -1, NULL, 0, NULL, NULL);
     char *file_nameA = HeapAlloc(GetProcessHeap(), 0, len);
-    Face *ret = NULL;
 
     WideCharToMultiByte(CP_UNIXCP, 0, file_name, -1, file_nameA, len, NULL, NULL);
     TRACE("looking for file %s name %s\n", debugstr_a(file_nameA), debugstr_w(face_name));
@@ -467,13 +466,15 @@ static Face *find_face_from_filename(con
                 file = face->file;
             else
                 file++;
-            if(!strcmp(file, file_nameA))
-                ret = face;
-            break;
+            if(!strcasecmp(file, file_nameA))
+            {
+                HeapFree(GetProcessHeap(), 0, file_nameA);
+                return face;
+            }
 	}
     }
     HeapFree(GetProcessHeap(), 0, file_nameA);
-    return ret;
+    return NULL;
 }
 
 static Family *find_family_from_name(const WCHAR *name)




More information about the wine-cvs mailing list