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

Optional array check mode #79

Closed
richfitz opened this issue Jul 31, 2017 · 3 comments
Closed

Optional array check mode #79

richfitz opened this issue Jul 31, 2017 · 3 comments
Projects

Comments

@richfitz
Copy link
Member

@richfitz richfitz commented Jul 31, 2017

To help with debugging it would be useful to compile in a "safe" mode where all array access is bounds checked. That would involve replacing all

x[i]

with an accessor function. This will have potentially very large performance costs so we'd want to be able to turn this on and off easily but probably at compile time

@richfitz
Copy link
Member Author

@richfitz richfitz commented Jan 11, 2018

See #49

@richfitz
Copy link
Member Author

@richfitz richfitz commented Jan 18, 2018

The use of size_t for i really exposes us to catastrophic underflows if we have i - 1 on array indexing...

@richfitz richfitz added this to In progress in development Jan 25, 2018
@richfitz
Copy link
Member Author

@richfitz richfitz commented Jan 25, 2018

Pending:

  • underflow
  • multidimensional array
  • error if used on non-array code
richfitz added a commit that referenced this issue Feb 1, 2018
@richfitz richfitz moved this from In progress to Done in development Feb 1, 2018
@richfitz richfitz closed this Feb 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
development
  
Done
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.