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

Runtime error when generating int32 values #98

Closed
peterjoel opened this Issue Apr 21, 2015 · 1 comment

Comments

Projects
None yet
2 participants
@peterjoel

peterjoel commented Apr 21, 2015

The int32 generator is currently unusable. It looks like the maxSize of int32 is too high (by 1), which causes an int check to fail.

Steps:

  1. Use code like:
     jsc.int32.generator(100)

Expected Result:
Should generate an integer

Actual Result:
There is a runtime exception: "random takes one or two integer arguments"

Stack Trace:
RC4.random @ rc4.js:97
randomInteger @ random.js:20
(anonymous function) @ primitive.js:55
(anonymous function) @ VM5720:1
InjectedScript._evaluateOn @ VM5277:883
InjectedScript._evaluateAndWrap @ VM5277:816
InjectedScript.evaluateOnCallFrame @ VM5277:942
RC4.random @ rc4.js:86
randomInteger @ random.js:20
(anonymous function) @ primitive.js:55
(anonymous function) @ VM5295:2
InjectedScript._evaluateOn @ VM5277:883
InjectedScript._evaluateAndWrap @ VM5277:816
InjectedScript.evaluate @ VM5277:682

phadej added a commit that referenced this issue Apr 23, 2015

@phadej

This comment has been minimized.

Show comment
Hide comment
@phadej

phadej Apr 23, 2015

Member

Weird. I'll fix this ASAP.

Member

phadej commented Apr 23, 2015

Weird. I'll fix this ASAP.

@phadej phadej added the bug label Apr 23, 2015

@phadej phadej added this to the 0.6.0 milestone Apr 23, 2015

@phadej phadej closed this in d3d4290 Apr 24, 2015

phadej added a commit that referenced this issue Apr 24, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment