[PATCH v3 7/9] dlls/oleaut32: Factor out TLB_get_funcdesc_by_name()

[Edward O'Callaghan]

From: Edward O'Callaghan

Signed-off-by: Edward O'Callaghan 
---
 dlls/oleaut32/typelib.c | 48 +++++++++++++++++++++++++----------------
 1 file changed, 29 insertions(+), 19 deletions(-)

diff --git a/dlls/oleaut32/typelib.c b/dlls/oleaut32/typelib.c
index 35c7af68efe..b86828c40c9 100644
--- a/dlls/oleaut32/typelib.c
+++ b/dlls/oleaut32/typelib.c
@@ -1677,6 +1677,18 @@ static inline TLBFuncDesc *TLB_get_funcdesc_by_memberid_invkind(ITypeInfoImpl *t
     return NULL;
 }

+static inline TLBFuncDesc *TLB_get_funcdesc_by_name(ITypeInfoImpl *typeinfo, const OLECHAR *name)
+{
+        for (int i = 0; i < typeinfo->typeattr.cFuncs; ++i) {
+            TLBFuncDesc *pFDesc = &typeinfo->funcdescs[i];
+
+            if (!lstrcmpiW(TLB_get_bstr(pFDesc->Name), name))
+                return pFDesc;
+        }
+
+        return NULL;
+}
+
 static inline TLBVarDesc *TLB_get_vardesc_by_memberid(ITypeInfoImpl *typeinfo, MEMBERID memid)
 {
     int i;
@@ -5033,24 +5045,24 @@ static HRESULT WINAPI ITypeLib2_fnIsName(
     *pfName=TRUE;
     for(tic = 0; tic < This->TypeInfoCount; ++tic){
         ITypeInfoImpl *pTInfo = This->typeinfos[tic];
+        TLBFuncDesc *pFDesc;
         TLBVarDesc *pVDesc;
-        UINT fdc;

if(!lstrcmpiW(TLB_get_bstr(pTInfo->Name), szNameBuf)) {
             TLB_set_bstr(szNameBuf, pTInfo->Name);
             goto ITypeLib2_fnIsName_exit;
         }

-        for(fdc = 0; fdc < pTInfo->typeattr.cFuncs; ++fdc) {
-            TLBFuncDesc *pFDesc = &pTInfo->funcdescs[fdc];
+        pFDesc = TLB_get_funcdesc_by_name(pTInfo, szNameBuf);
+        if (pFDesc) {
+            TLB_set_bstr(szNameBuf, pFDesc->Name);
+            goto ITypeLib2_fnIsName_exit;
+        }

-            if(!lstrcmpiW(TLB_get_bstr(pFDesc->Name), szNameBuf)) {
-                TLB_set_bstr(szNameBuf, pFDesc->Name);
-                goto ITypeLib2_fnIsName_exit;
-            }
+        for(UINT fdc = 0; fdc < pTInfo->typeattr.cFuncs; ++fdc) {
+            pFDesc = &pTInfo->funcdescs[fdc];

-            int pc;
-            for(pc=0; pc < pFDesc->funcdesc.cParams; pc++){
+            for(int pc=0; pc < pFDesc->funcdesc.cParams; pc++){
                 if(!lstrcmpiW(TLB_get_bstr(pFDesc->pParamDesc[pc].Name), szNameBuf)) {
                     TLB_set_bstr(szNameBuf, pFDesc->pParamDesc[pc].Name);
                     goto ITypeLib2_fnIsName_exit;
@@ -5100,7 +5112,8 @@ static HRESULT WINAPI ITypeLib2_fnFindName(
     // TODO: factor out common impl with fnIsName().
     for(tic = 0; count < *found && tic < This->TypeInfoCount; ++tic) {
         ITypeInfoImpl *pTInfo = This->typeinfos[tic];
-        UINT fdc;
+        TLBFuncDesc *pFDesc;
+        TLBVarDesc *pVDesc;

if(!lstrcmpiW(TLB_get_bstr(pTInfo->Name), szNameBuf)) {
             memid[count] = MEMBERID_NIL;
@@ -5108,17 +5121,14 @@ static HRESULT WINAPI ITypeLib2_fnFindName(
             goto ITypeLib2_fnFindName_exit;
         }

-        for(fdc = 0; fdc < pTInfo->typeattr.cFuncs; ++fdc) {
-            TLBFuncDesc *pFDesc = &pTInfo->funcdescs[fdc];
-
-            if(!lstrcmpiW(TLB_get_bstr(pFDesc->Name), szNameBuf)) {
-                memid[count] = pFDesc->funcdesc.memid;
-                TLB_set_bstr(szNameBuf, pFDesc->Name);
-                goto ITypeLib2_fnFindName_exit;
-            }
+        pFDesc = TLB_get_funcdesc_by_name(pTInfo, szNameBuf);
+        if (pFDesc) {
+            memid[count] = pFDesc->funcdesc.memid;
+            TLB_set_bstr(szNameBuf, pFDesc->Name);
+            goto ITypeLib2_fnFindName_exit;
         }

-        TLBVarDesc *pVDesc = TLB_get_vardesc_by_name(pTInfo, szNameBuf);
+        pVDesc = TLB_get_vardesc_by_name(pTInfo, szNameBuf);
         if (pVDesc) {
             memid[count] = pVDesc->vardesc.memid;
             TLB_set_bstr(szNameBuf, pVDesc->Name);