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

Simplified API for copying to/from page locked memory. #697

Merged
merged 7 commits into from
Jan 9, 2024

Conversation

MoFtZ
Copy link
Collaborator

@MoFtZ MoFtZ commented Nov 24, 2021

Fixes #691.

Removes the various CopyFrom/CopyToPageLockedAsync methods in favour of the more generic CopyTo/CopyFrom ArrayView methods. This allows copying to/from a subset of the page locked array without adding a lot of extra to methods specifically support it.

PageLockedScoped now exposes an ArrayView property that can be used to interact with the existing CopyFrom/CopyTo ArrayView methods.

@MoFtZ MoFtZ mentioned this pull request Nov 24, 2021
m4rs-mt
m4rs-mt previously approved these changes Nov 24, 2021
@m4rs-mt m4rs-mt added this to the v2.0 milestone Nov 30, 2021
@m4rs-mt m4rs-mt added enhancement feature A new feature (or feature request) labels Jan 20, 2022
@MoFtZ MoFtZ force-pushed the feature/simplify-page-lock branch 2 times, most recently from 3e22676 to 9e35572 Compare December 5, 2022 04:53
@MoFtZ MoFtZ force-pushed the feature/simplify-page-lock branch from 9e35572 to a87266b Compare February 23, 2023 08:00
@m4rs-mt
Copy link
Owner

m4rs-mt commented Aug 29, 2023

@MoFtZ looks like we have to address some conflicts before merging this feature.

@MoFtZ MoFtZ force-pushed the feature/simplify-page-lock branch from a87266b to 955aaee Compare August 31, 2023 11:39
@MoFtZ MoFtZ force-pushed the feature/simplify-page-lock branch from 1a17bef to d4563b5 Compare January 8, 2024 11:35
@m4rs-mt m4rs-mt force-pushed the feature/simplify-page-lock branch from d4563b5 to db43cf8 Compare January 8, 2024 14:24
@MoFtZ MoFtZ merged commit 5ac2165 into m4rs-mt:master Jan 9, 2024
44 checks passed
@MoFtZ MoFtZ deleted the feature/simplify-page-lock branch January 9, 2024 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature A new feature (or feature request)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PageLockScope issue
2 participants