-
-
Notifications
You must be signed in to change notification settings - Fork 888
Change size guards for WrapMemory to allow Memory/IMemoryOwner blocks to be larger than the image size #1526
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
Change size guards for WrapMemory to allow Memory/IMemoryOwner blocks to be larger than the image size #1526
Conversation
… larger than the required image size.
Codecov Report
@@ Coverage Diff @@
## master #1526 +/- ##
=======================================
Coverage 83.50% 83.50%
=======================================
Files 742 742
Lines 32801 32801
Branches 3671 3671
=======================================
Hits 27392 27392
Misses 4691 4691
Partials 718 718
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
Change size guards for WrapMemory to allow Memory/IMemoryOwner blocks to be larger than the image size
Prerequisites
Description
Changes memory length guards in Image.WrapMemory() methods to allow passing blocks that are larger than the image size. Fixes #1522.