[Monetdb-developers] MonetDB: gdk-calc - gdk_calc.c, mod_typeswitchloop(): initialize...

Sjoerd Mullender sjoerd at acm.org
Thu Feb 16 09:16:43 CET 2012


On 2012-02-15 20:12, Stefan Manegold wrote:
> Changeset: fc0d3fad41c2 for MonetDB
> URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=fc0d3fad41c2
> Modified Files:
> 	gdk/gdk_calc.c
> Branch: gdk-calc
> Log Message:
> 
> gdk_calc.c, mod_typeswitchloop(): initialize "nils" (hopefully correctly)

Which sucky compiler was this?  There is no code path where nils was not
initialized.  (This also means that the value you choose here doesn't
matter.)

> 
> diffs (12 lines):
> 
> diff --git a/gdk/gdk_calc.c b/gdk/gdk_calc.c
> --- a/gdk/gdk_calc.c
> +++ b/gdk/gdk_calc.c
> @@ -6157,7 +6157,7 @@ mod_typeswitchloop(const void *lft, int 
>  		   void *dst, int tp, BUN cnt,
>  		   int abort_on_error, const char *func)
>  {
> -	BUN nils;
> +	BUN nils = BUN_NONE;
>  
>  	switch (ATOMstorage(tp1)) {
>  	case TYPE_bte:
> _______________________________________________
> Checkin-list mailing list
> Checkin-list at monetdb.org
> http://mail.monetdb.org/mailman/listinfo/checkin-list


-- 
Sjoerd Mullender

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 372 bytes
Desc: OpenPGP digital signature
URL: <http://www.monetdb.org/pipermail/developers-list/attachments/20120216/c8874a3a/attachment.sig>


More information about the developers-list mailing list