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

Martin Kersten martin.kersten at cwi.nl
Mon Nov 7 15:11:51 CET 2016


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 at monetdb.org
> https://www.monetdb.org/mailman/listinfo/checkin-list
>



More information about the developers-list mailing list