Re: MonetDB: malupgrade - Clear variables before resetting mb->vtop.

WOW .... that took me quite some time nailing the example and looking into the wrong direction thx

On 07/11/16 12:38, Sjoerd Mullender wrote:

Changeset: 8ac922886df2 for MonetDB URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=8ac922886df2 Modified Files: monetdb5/mal/mal_instruction.c Branch: malupgrade Log Message:

Clear variables before resetting mb->vtop.

diffs (20 lines):

diff --git a/monetdb5/mal/mal_instruction.c b/monetdb5/mal/mal_instruction.c --- a/monetdb5/mal/mal_instruction.c +++ b/monetdb5/mal/mal_instruction.c @@ -197,13 +197,13 @@ freeMalBlk(MalBlkPtr mb) mb->stmt[i] = NULL; } mb->stop = 0;

• for(i=0; i< mb->vtop; i++)

• if (isVarConstant(mb, i))
  

• 	VALclear(&getVarConstant(mb,i));
  

  mb->vtop = 0; mb->vid = 0; GDKfree(mb->stmt); mb->stmt = 0;

• for(i=0; i< mb->vtop; i++)
• if (isVarConstant(mb, i))

• VALclear(&getVarConstant(mb,i));
  

  GDKfree(mb->var); mb->var = 0;

---

checkin-list mailing list checkin-list@monetdb.org https://www.monetdb.org/mailman/listinfo/checkin-list