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

Ix constants and compile-time errors #10

Open
leepike opened this issue Feb 1, 2014 · 3 comments
Open

Ix constants and compile-time errors #10

leepike opened this issue Feb 1, 2014 · 3 comments

Comments

@leepike
Copy link
Contributor

leepike commented Feb 1, 2014

https://github.com/agacek noted that constants outside the range of an Ix n type are not a compile-time error and are simply moded. Perhaps a compile-time error should be thrown here?

@elliottt
Copy link
Contributor

elliottt commented Feb 4, 2014

Maybe this is a good opportunity for model checker annotations?

On Sat, Feb 1, 2014 at 11:43 AM, Lee Pike notifications@github.com wrote:

https://github.com/agacek noted that constants outside the range of an Ix
n type are not a compile-time error and are simply moded. Perhaps a
compile-time error should be thrown here?

Reply to this email directly or view it on GitHubhttps://github.com//issues/10
.

@Hodapp87
Copy link
Contributor

I have noted the same when dealing with arrays, and it seems like modulo could introduce some unexpected behavior.

@leepike
Copy link
Contributor Author

leepike commented Oct 24, 2016

Updating the issue: unless there are objections, this should be implemented.

Note: the code is still safe with the modulus, but there may be surprises for the user.

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

3 participants