Bugs item #2907463, was opened at 2009-12-02 14:12
Message generated for change (Comment added) made by stmane
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=2907463&group_id=56967

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Build
Group: MonetDB5 CVS Head
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Stefan Manegold (stmane)
Assigned to: Tang Nan (ntang928)
Summary: M5: pma.mx fails to compile on Windows

Initial Comment:
On Windows, the Microsoft compiler correctly(!) complains about various 
type-inconsistencies that might lead to overflows and hence unexpected or 
incorrect behaviors on all platforms.

For details see
http://monetdb.cwi.nl/testing/projects/monetdb/Current/MonetDB5/.Mic.32.32.d.1-Windows6.0..2009.12.02_07-00-56/make.out.html
http://monetdb.cwi.nl/testing/projects/monetdb/permastore/2009-12-02/Current/make-MonetDB5-Mic.32.32.d.1-Windows6.0.errors.html#l354
http://monetdb.cwi.nl/testing/projects/monetdb/Current/MonetDB5/.Mic.64.32.d.1-Windows6.0..2009.12.02_07-00-56/make.out.html
http://monetdb.cwi.nl/testing/projects/monetdb/permastore/2009-12-02/Current/make-MonetDB5-Mic.64.32.d.1-Windows6.0.errors.html#l353
http://monetdb.cwi.nl/testing/projects/monetdb/Current/MonetDB5/.Mic.64.64.d.1-Windows6.0..2009.12.02_07-00-56/make.out.html
http://monetdb.cwi.nl/testing/projects/monetdb/permastore/2009-12-02/Current/make-MonetDB5-Mic.64.64.d.1-Windows6.0.errors.html#l353

See also
ID: 2890702 "M5: pma.mx fails to compile on various platforms"
https://sourceforge.net/tracker/?func=detail&aid=2890702&group_id=56967&atid=482468
and (e.g.)
for i in 2 3 4 5 6 7 ; do ( cvs log -Nr1.$i src/modules/mal/pma.mx ; cvs diff 
-r1.{$[i-1],$i} src/modules/mal/pma.mx ) | cdiff ; done

Do not hesitate to ask in case you have any questions about proper and portable 
type usage in MonetDB.


----------------------------------------------------------------------

>Comment By: Stefan Manegold (stmane)
Date: 2010-03-17 20:10

Message:
compilation appears to have been fixed by
===================================================================
2009/12/04 - ntang928: MonetDB5/src/modules/mal/pma.mx,1.8
fix some bugs about type conversion failure on Windows
===================================================================
2009/12/05 - ntang928: MonetDB5/src/modules/mal/pma.mx,1.9
minor revision for type conversion and error message processing
===================================================================
2009/12/06 - ntang928: MonetDB5/src/modules/mal/pma.mx,1.10
for correcting type conversion warnings
===================================================================

However, in case the code should ever be used, it should be double checked
for type consistency.
E.g., type BUN is not equal to type wrd, and dereferencing a pointer to a
wrd as a BUN will not alwasy result in what one might expect, surely not
with 32-bit OIDs on a 64-bit big-endian system ...

Closing this one.

I'll whether I understand the code well enough to take care of the type
issue(s) ...


----------------------------------------------------------------------

Comment By: Martin Kersten (mlkersten)
Date: 2010-03-17 19:46

Message:
Is this issue still present? Or can it be closed.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=2907463&group_id=56967

------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Monetdb-bugs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-bugs

Reply via email to