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

Read from files concurrently #9

Closed
mvdan opened this issue Jun 15, 2015 · 1 comment
Closed

Read from files concurrently #9

mvdan opened this issue Jun 15, 2015 · 1 comment

Comments

@mvdan
Copy link
Owner

mvdan commented Jun 15, 2015

As suggested by @bep, we could read and run the regex over files passed as arguments concurrently all at once, instead of one after the other. For regular files this doesn't make much sense in the general case, but it could make sense in files that cause blocking reads like named pipes or stuff that goes over the network.

The only downside I can see to this is that it's a bit overkill for the generic, simple case.

@mvdan
Copy link
Owner Author

mvdan commented Jun 20, 2015

I'm not going to implement this. For edge cases where you want it to be asynchronous among files, you can fire up multiple processes. This way we don't overcomplicate the generic use case.

@mvdan mvdan closed this as completed Jun 20, 2015
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

1 participant