Opened 7 years ago
Closed 22 months ago
#1520 closed defect (fixed)
binary-search: test failure: (=) bad argument type - not a number: #f
| Reported by: | Mario Domenech Goulart | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | 5.4 | 
| Component: | extensions | Version: | 5.0.0 | 
| Keywords: | binary-search | Cc: | |
| Estimated difficulty: | 
Change History (6)
comment:1 Changed 4 years ago by
comment:2 Changed 2 years ago by
| Milestone: | someday → 5.4 | 
|---|
comment:3 Changed 23 months ago by
| Resolution: | → fixed | 
|---|---|
| Status: | new → closed | 
comment:4 Changed 23 months ago by
| Resolution: | fixed | 
|---|---|
| Status: | closed → reopened | 
comment:5 Changed 23 months ago by
I think the problem is that (square x) returns bignums on 32-bit platforms, while binary-search assumes fixnums everywhere.
The code checks that the sequence is within fixnum range, but it doesn't do so for the return value of proc in the loop (but still calls fixnum ops on it!)
comment:6 Changed 22 months ago by
| Resolution: | → fixed | 
|---|---|
| Status: | reopened → closed | 
The issue been fixed with the 0.2 release (http://bugs.call-cc.org/changeset/43071/project).  After that, tests succeed: http://salmonella-linux-x86.call-cc.org/master/clang/linux/x86/2023/12/27/salmonella-report/test/binary-search.html
Note: See
        TracTickets for help on using
        tickets.
    

More recent test failure: http://salmonella-linux-x86.call-cc.org/master/clang/linux/x86/2021/08/27/salmonella-report/test/binary-search.html