[PATCH v5 07/10] jscript: Detach the frame's scope before inserting eval() variables into it.

[Gabriel Ivăncescu]

From: Gabriel Ivăncescu

This is no-op now, but is needed when we use specialized detached locals
buffer instead of the variable obj.

Signed-off-by: Gabriel Ivăncescu 
---
 dlls/jscript/engine.c | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/dlls/jscript/engine.c b/dlls/jscript/engine.c
index d5659d55f17..0b044076676 100644
--- a/dlls/jscript/engine.c
+++ b/dlls/jscript/engine.c
@@ -3343,7 +3343,11 @@ HRESULT exec_source(script_ctx_t *ctx, DWORD flags, bytecode_t *bytecode, functi
     }

if((flags & EXEC_EVAL) && ctx->call_ctx) {
-        variable_obj = jsdisp_addref(ctx->call_ctx->variable_obj);
+        variable_obj = ctx->call_ctx->variable_obj;
+        hres = detach_variable_object(ctx, ctx->call_ctx, FALSE);
+        if(FAILED(hres))
+            return hres;
+        jsdisp_addref(variable_obj);
     }else if(!(flags & (EXEC_GLOBAL | EXEC_EVAL))) {
         variable_obj = NULL;  /* set to scope's dispex later */
     }else if(bytecode->named_item) {
@@ -3396,12 +3400,6 @@ HRESULT exec_source(script_ctx_t *ctx, DWORD flags, bytecode_t *bytecode, functi
             this_obj = NULL;
     }

-    if(ctx->call_ctx && (flags & EXEC_EVAL)) {
-        hres = detach_variable_object(ctx, ctx->call_ctx, FALSE);
-        if(FAILED(hres))
-            goto fail;
-    }
-
     frame = calloc(1, sizeof(*frame));
     if(!frame) {
         hres = E_OUTOFMEMORY;