[Monetdb-developers] [Monetdb-pf-checkins] pathfinder/modules/pftijah pftijah.mx, 1.123, 1.124

Stefan Manegold Stefan.Manegold at cwi.nl
Tue May 29 16:56:44 CEST 2007


On Tue, May 29, 2007 at 04:09:27PM +0200, Jan Flokstra wrote:
[...]
> We were aware of this tail type and that's why we created the cast. Henning 
> was a bit hesitant to change the bat type to [oid,oid] because he expected  
> problems with '<' and '>' so we decided to create the extra cast.

What kind of problems do you mean?
Clean and proper use of types should prevent such problems;
"arbitrary" casting calls for such (and other) problems.
Hence, choose the proper types at all places, and cast as little as
possible.
(just my humble advice)

Stefan

> The value can never be negative so we never checked for this. We could
> make it an assert();
> Furthermore the text //64bit ERROR was to remind us of  our own error on 64bit 
> oid arch's and not an error of monetdb.
> 
> JanF.
[...]

-- 
| Dr. Stefan Manegold | mailto:Stefan.Manegold at cwi.nl |
| CWI,  P.O.Box 94079 | http://www.cwi.nl/~manegold/  |
| 1090 GB Amsterdam   | Tel.: +31 (20) 592-4212       |
| The Netherlands     | Fax : +31 (20) 592-4312       |




More information about the developers-list mailing list