[Monetdb-developers] [Monetdb-pf-checkins] pathfinder/modules/pftijah pftijah.mx, 1.123, 1.124
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
(just my humble advice)
> 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.
| 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