Skip to content

OpenSpin accepts constant values larger than 32 bits #23

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.

Sign up for GitHub

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

Jump to bottom
Closed
bweir opened this issue Jan 15, 2016 · 2 comments
Closed

OpenSpin accepts constant values larger than 32 bits #23

bweir opened this issue Jan 15, 2016 · 2 comments
Assignees
@reltham
Labels
bug

Comments

@bweir
Copy link

bweir commented Jan 15, 2016

in PropTool, when constants exceed the maximum number of bits, the compiler complains.

OpenSpin, it quietly accepts it even though the result is undefined. I played around with it and discovered that OpenSpin will apparently accept values of any size, no matter how large, like $111111111100000000000000000000000000.
@reltham reltham self-assigned this Jan 15, 2016
@reltham reltham added the bug label Jan 15, 2016
reltham added a commit that referenced this issue Jan 16, 2016
@reltham
Fix for Issue #23
2e30869 
Fix for issue #23  where it was silently accepting constants larger than
32bits instead of reporting an error.
Updated copyright line to 2016.
@reltham
Copy link
Collaborator

reltham commented Jan 17, 2016

Fix contained in Release 1.00.78

@reltham reltham closed this as completed Jan 17, 2016
@bweir
Copy link
Author

bweir commented Jan 17, 2016

Thanks!

On Sat, Jan 16, 2016 at 4:09 PM, Roy Eltham notifications@github.com
wrote:

Closed #23 #23.

—
Reply to this email directly or view it on GitHub
#23 (comment).

Brett Weir, Founder

Engineering Made Awesome
310.245.1775 • brett@lamestation.com • lamestation.com

@reltham reltham mentioned this issue Feb 26, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@reltham reltham

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@reltham @bweir