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

Stream sane_read #19

Open
Jupi007 opened this issue Nov 4, 2024 · 2 comments
Open

Stream sane_read #19

Jupi007 opened this issue Nov 4, 2024 · 2 comments

Comments

@Jupi007
Copy link
Collaborator

Jupi007 commented Nov 4, 2024

Stream sane_read() response instead of needing to call this method repetitively.

@Craftplacer
Copy link
Collaborator

Craftplacer commented Nov 4, 2024

I've been asking myself how much "cleaner" we should present the developer with options.

Should be there a Stream<Uint8List> Sane.scan(Function(Options options) onConfigure) method instead?

If yes, should we still expose start() set_io_mode() cancel() to the developer?

@Jupi007
Copy link
Collaborator Author

Jupi007 commented Nov 4, 2024

Imo, this and #18 should be part of an higher level scanner.dart library.
In sane.dart we expose the low level pure dart implementation, on top of what we can build something with a better oop approach.
sane.dart should stay as simple and as close as possible to the C library, so we can test it easily.

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