[PATCH 09/10] jscript: Move filling the PROTREF into a helper.

[Gabriel Ivăncescu]

From: Gabriel Ivăncescu

Signed-off-by: Gabriel Ivăncescu 
---
 dlls/jscript/dispex.c | 34 +++++++++++++++++++++-------------
 1 file changed, 21 insertions(+), 13 deletions(-)

diff --git a/dlls/jscript/dispex.c b/dlls/jscript/dispex.c
index 7fa80fe51cd..d17e0c04228 100644
--- a/dlls/jscript/dispex.c
+++ b/dlls/jscript/dispex.c
@@ -872,9 +872,27 @@ static HRESULT fill_props(jsdisp_t *obj)
     return S_OK;
 }

-static HRESULT fill_protrefs(jsdisp_t *This)
+static HRESULT fill_protref(jsdisp_t *This, unsigned hash, const WCHAR *name, DWORD ref)
 {
     struct prop_desc prop;
+    HRESULT hres;
+
+    hres = find_prop_name(This, hash, name, FALSE, &prop);
+    if(hres != S_FALSE)
+        return hres;
+    if(is_dispex_prop_id(prop.id)) {
+        dispex_prop_t *p = &This->props[prop_id_to_idx(prop.id)];
+        p->type = PROP_PROTREF;
+        p->flags = 0;
+        p->u.ref = ref;
+    }else if(!alloc_protref(This, name, ref)) {
+        return E_OUTOFMEMORY;
+    }
+    return S_OK;
+}
+
+static HRESULT fill_protrefs(jsdisp_t *This)
+{
     dispex_prop_t *iter;
     HRESULT hres;

@@ -890,19 +908,9 @@ static HRESULT fill_protrefs(jsdisp_t *This)
         return hres;

for(iter = This->prototype->props; iter < This->prototype->props+This->prototype->prop_cnt; iter++) {
-        hres = find_prop_name(This, iter->hash, iter->name, FALSE, &prop);
-        if(FAILED(hres))
+        hres = fill_protref(This, iter->hash, iter->name, iter - This->prototype->props);
+        if(hres != S_OK)
             return hres;
-        if(hres != S_OK) {
-            if(is_dispex_prop_id(prop.id)) {
-                dispex_prop_t *p = &This->props[prop_id_to_idx(prop.id)];
-                p->type = PROP_PROTREF;
-                p->flags = 0;
-                p->u.ref = iter - This->prototype->props;
-            }else if(!alloc_protref(This, iter->name, iter - This->prototype->props)) {
-                return E_OUTOFMEMORY;
-            }
-        }
     }

return S_OK;