Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fxreverse-fixnum error in self call #787

Open
ghost opened this issue Mar 26, 2017 · 3 comments
Open

fxreverse-fixnum error in self call #787

ghost opened this issue Mar 26, 2017 · 3 comments

Comments

@ghost
Copy link

ghost commented Mar 26, 2017

In r6rs/arithmetic/fixnums.body.scm there is an error on line 449, in the function fxreverse-fixnum:

(+ 1 (fxreverse-fixnum (- fx (least-fixnum))))

There should be a second parameter in the call to fxreverse-fixnum.

Should the line be: (+ 1 (fxreverse-fixnum (- fx (least-fixnum)) k))

@WillClinger
Copy link
Member

Thank you.

This is in tools/R6RS/r6rs/arithmetic/fixnums.body.scm

Fixed in changeset 52d391e

I'm leaving this open for now because the fix needs to be tested further and pushed to Snow.

@WillClinger WillClinger modified the milestones: Larceny 1.5, Larceny 1.3 Jul 1, 2017
@WillClinger
Copy link
Member

Changing the milestone because v1.3 doesn't need to wait on this.

@WillClinger
Copy link
Member

I intended only to change the milestone, not to close the issue.

@WillClinger WillClinger reopened this Jul 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant