Skip to content

Optimize temp files removal #14

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

Closed
xonixx opened this issue Jun 14, 2023 · 1 comment
Closed

Optimize temp files removal #14

xonixx opened this issue Jun 14, 2023 · 1 comment
Assignees
Milestone

Comments

@xonixx
Copy link
Owner

xonixx commented Jun 14, 2023

Now synthetic test is ~twice slower compared to tush.

$ time tush-check 1000.tush 

real    0m1.055s
user    0m0.787s
sys     0m0.180s
$ time ./fhtagn.awk 1000.tush 

real    0m1.982s
user    0m1.765s
sys     0m0.294s

This is due to calling system("rm -f " stdOutF " " stdErrF) for every test. Can we call rm one time for all temp files at the end?

@xonixx xonixx added this to the 0.1.0 milestone Jun 14, 2023
@xonixx xonixx self-assigned this Jun 14, 2023
xonixx added a commit that referenced this issue Jun 15, 2023
@xonixx xonixx closed this as completed Jun 15, 2023
xonixx added a commit that referenced this issue Jun 15, 2023
@xonixx
Copy link
Owner Author

xonixx commented Jun 15, 2023

Ok, now it's faster.

$ time ./fhtagn.awk 1000.tush 

real    0m0.841s
user    0m0.728s
sys     0m0.146s

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