Why not fix this "properly" by changing the type of cnt (and the
function return value and several other variables)?
At the very least you could have an assert to check that the value fits
in an int.

On 2010-06-10 10:54, Fabian Groffen wrote:
> Changeset: 820a7637cf5e for MonetDB
> URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=820a7637cf5e
> Modified Files:
>       MonetDB5/src/optimizer/opt_mapreduce.mx
> Branch: default
> Log Message:
> 
> fix compilation failure opt_mapreduce.mx:141: '=' : conversion from 'size_t' 
> to 'int', possible loss of data
> 
> 
> diffs (12 lines):
> 
> diff -r c7d1cd0d0bbb -r 820a7637cf5e MonetDB5/src/optimizer/opt_mapreduce.mx
> --- a/MonetDB5/src/optimizer/opt_mapreduce.mx Wed Jun 09 15:30:45 2010 +0200
> +++ b/MonetDB5/src/optimizer/opt_mapreduce.mx Thu Jun 10 10:54:23 2010 +0200
> @@ -138,7 +138,7 @@
>               return 0;
>       }
>       cloud = BATdescriptor(bid);
> -     cnt = BATcount(cloud);
> +     cnt = (int)BATcount(cloud);
>       BBPreleaseref(bid); /* we're done with it */
>       return(cnt);
>  }
> _______________________________________________
> Checkin-list mailing list
> checkin-l...@monetdb.org
> http://mail.monetdb.org/mailman/listinfo/checkin-list


-- 
Sjoerd Mullender

Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate 
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the 
lucky parental unit.  See the prize list and enter to win: 
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
Monetdb-developers mailing list
Monetdb-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/monetdb-developers

Reply via email to