OCaml Forge

Detail: [#1285] test/test.ml does not compile on 32 bit

Bugs: Browse | Download .csv | Monitor

[#1285] test/test.ml does not compile on 32 bit

Date:
2013-06-07 09:28
Priority:
3
State:
Closed
Submitted by:
Hendrik Tews (hendrik_tews)
Assigned to:
Nobody (None)
Resolution:
Fixed
Severity:
None
Hardware:
None
 
Operating System:
Version:
 
URL:
Summary:
test/test.ml does not compile on 32 bit

Detailed description
Hi,

the part after

if Sys.word_size = 64 then
begin

in test/test.ml from line 306 does not compile on 32 bit
platforms. The integer constants there are too large. I would
suggest using (Int64.to_int 0x...L) on the right hand side. Then
the test should even run wit Sys.word_size = 32.

Bye,

Hendrik Tews

Followup

Message
Date: 2013-06-11 20:21
Sender: ygrek

Yes, the tracker is weird :)
But I can see your attachment. Applied, thank you!
Date: 2013-06-11 18:16
Sender: Hendrik Tews

... and it even displays my followups in reverse order !!
Date: 2013-06-11 18:14
Sender: Hendrik Tews

this tracker does not accept my attachments, please fix the tracker or get the patch from the debian package
Date: 2013-06-11 18:12
Sender: Hendrik Tews

with this patch the test compile and run on amd64 and i386

Attached Files:

Attachments:
fix-text-32-bit.patch

Changes:

Field Old Value Date By
status_idOpen2013-06-11 20:21ygrek
close_dateNone2013-06-11 20:21ygrek
ResolutionNone2013-06-11 20:21ygrek
File Added213: fix-text-32-bit.patch2013-06-11 18:12hendrik_tews