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

Fix a Useless Use Of Cat in tools/measure.sh #485

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dyyim741
Copy link
Contributor

With the UUoC version, cat has to read the file into memory, then write it out to the pipe, and the command has to read the data from the pipe, so the kernel has to copy the whole file three times whereas in the redirected case, the kernel only has to copy the file once. It is quicker to do something once than to do it three times.

Ref : http://porkmail.org/era/unix/award.html

Signed-off-by: DaYea Yim <dyyim@protonmail.com>
@clover2123
Copy link
Contributor

@dyyim741 Sorry for late review. Unfortunately we decide to clear up some redundant files as described in #499 and your commits including #486 will be no longer valid. But we always welcome any suggestion. Please feel free to participate in this project. Thanks for your contribution.

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

Successfully merging this pull request may close these issues.

2 participants