-
Notifications
You must be signed in to change notification settings - Fork 28
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
API for Buffer #202
Comments
Thank you for making this list as I don't understand the internals of Automa. I think it is fine to keep the Also, do you know if Automa is using the Lastly, does Automa use any of the |
Automa does not use From what I can tell, what Automa uses is:
|
I was taking with @nhz2 about how Automa (and presumably other packages) use internals from TranscodingStreams, and how it would be nice to make it stop doing that. However, TS was created by the same author as Automa, and the two packages' internals are entangled. So, for Automa to not rely on TS internals, TS needs to offer these things as API. In particular:
.data
field of aTranscodingStream
, either as aVector{UInt8}
or potentiallyMemory{UInt8}
going forward.markpos
and.bufferpos
fields of the readBuffer
of a stream.marginpos
of the readBuffer
of a streamIdeally, the
Buffer
object can also be obtained so the user can manipulate theBuffer
directly.Another approach is for Automa to stop using TranscodingStreams, and instead use something like BufferedStreams.jl. However, I believe TranscodingStreams might have originally been implemented to support Automa.
The text was updated successfully, but these errors were encountered: