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

Add runtime debug testing with appropriate compile flag #3

Open
tiehuis opened this issue Feb 14, 2017 · 0 comments
Open

Add runtime debug testing with appropriate compile flag #3

tiehuis opened this issue Feb 14, 2017 · 0 comments
Assignees

Comments

@tiehuis
Copy link
Owner

tiehuis commented Feb 14, 2017

This is a small addition that really helps when ensuring that one has written the generation steps correctly.

We can perform the following:

  • Ensure map and arr calls have the correct number of items within them
  • Ensure buffer overflows do not occur

These are not foolproof, especially with buffer overflows as we would need to consider the case where the values inserted are the largest possible and then check that, but it should be a reasonably good first step.

For statically assuring the buffers are large enough, we could provide a macro, say np_size_req which expects a list of np calls and returns the required space needed in the worst case. May not be tremendously helpful though.

@tiehuis tiehuis self-assigned this Feb 14, 2017
tiehuis added a commit that referenced this issue Feb 15, 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