ublas icon indicating copy to clipboard operation
ublas copied to clipboard

Added out of range exception for static tensor's

Open shikharvashistha opened this issue 4 years ago • 2 comments

This pulls adds out of range exception for static tensors

Fixes : #117

shikharvashistha avatar Nov 05 '21 08:11 shikharvashistha

This Pull request Passed all of clang-tidy tests. :+1:

github-actions[bot] avatar Nov 05 '21 09:11 github-actions[bot]

I think you misconstrued the issue. The arity of the function is already checked using static assert and on top of that, you added the same check during runtime. The issue was to fix the out of bound access, for example:

int a[20];
a[0] = 3; // valid access
a[23] = 5; // invalid access or accessing unowned memory, which is the out of bound access.

amitsingh19975 avatar Feb 07 '22 12:02 amitsingh19975