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

Change system and pointer APIs to pervasively use the Range type #132

Open
k-sareen opened this issue Feb 17, 2023 · 2 comments
Open

Change system and pointer APIs to pervasively use the Range type #132

k-sareen opened this issue Feb 17, 2023 · 2 comments

Comments

@k-sareen
Copy link
Contributor

Currently APIs such as Ptr.atContents and System.file{Read,Write}K assume the input will be an Array<byte>. Changing to a Range<byte> would make it more general.

@titzer
Copy link
Owner

titzer commented Mar 27, 2023

I have done some work on this recently, e.g. Pointer.atContents can work with ranges now.

For the built-in System component, I was thinking of adding new methods that take a Range<byte> and then deprecating the old ones. This requires a stable rev, as they must first be introduced into v3i and then (with the Pointer.atContents change above integrated into stable) added the native implementations after the rev.

@k-sareen
Copy link
Contributor Author

Yes that makes sense. Sorry for being AWOL again -- just have too many deadlines. Will be more free after mid-April. Might try my hand at making an LSP server again.

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