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

Initializing a gr_mat with -1 rows and -1 columns succeeds and allocates one entry #2235

Open
rburing opened this issue Feb 3, 2025 · 1 comment

Comments

@rburing
Copy link
Contributor

rburing commented Feb 3, 2025

Probably there should be an error/crash instead, when either the number of rows or the number of columns is negative.

The same applies to (all?) other matrix types.

@albinahlback
Copy link
Collaborator

Or clarify in the documentation that we assume that the number of rows and columns are valid.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants