compiler: Fix for window.anchor.left

- Fixed a bug in _FixRefExp causing deep properties to be uninitialized
- This fixed condition in DataBox::OnRedraw and potential overflow on huge IdList

diff --git a/compiler/bootstrap/libec/bootstrap/pass2.c b/compiler/bootstrap/libec/bootstrap/pass2.c
index 7134c61..8c7194d 100644 (file)
--- a/compiler/bootstrap/libec/bootstrap/pass2.c
+++ b/compiler/bootstrap/libec/bootstrap/pass2.c
@@ -1109,7 +1109,7 @@ bracketExp->list = (((void *)0));
 __ecereMethod___ecereNameSpace__ecere__sys__OldList_Remove((&*exp->list), idExp);
 __ecereMethod___ecereNameSpace__ecere__sys__OldList_Add((&*exp->list), newExp);
 FreeExpression(bracketExp);
-*expPtr = newExp;
+*expPtr = exp;
 }
 }
 else if(*expPtr && (*expPtr)->type == 4 && (*expPtr)->op.op == '&' && !(*expPtr)->op.exp1 && memberExp && (memberExp->type == 5 || memberExp->type == 34) && memberExp->list && (*memberExp->list).count > 1)

diff --git a/compiler/libec/src/pass2.ec b/compiler/libec/src/pass2.ec
index 8b5053a..928d8a9 100644 (file)
--- a/compiler/libec/src/pass2.ec
+++ b/compiler/libec/src/pass2.ec
@@ -38,7 +38,7 @@ static void _FixRefExp(Expression * expPtr, Expression * memberExpPtr)
          exp.list->Add(newExp);
          FreeExpression(bracketExp);
 
-         *expPtr = newExp; //FixRefExp(newExp);
+         *expPtr = exp; //FixRefExp(newExp);      // TESTING THIS: exp was not used!
       }
    }
    else if(*expPtr && (*expPtr).type == opExp && (*expPtr).op.op == '&' && !(*expPtr).op.exp1 &&