Speed up last_status by avoiding wc -l #3
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
On my system using native bash functions instead of wc -l speeds up the last_status by about 0.2 seconds. This may be because windows pipes are slow or some issue with wc. Before: ``` master last_status: 0.463 user_info: 0.066 cwd: 0.071 scm: 2.549 ``` After: ``` last_status: 0.228 user_info: 0.079 cwd: 0.073 scm: 2.714 ``` Performance numbers aren't exact as they vary a bit, but you can see a bit difference in the runtime for last_status.
Last status: get read to bail out earlier. Get jobs command to output less by only outputting the pids of stopped jobs.
Speed up last_status further by not using $() around job collection.
Small adjustment to unbreak this :)
AlexiTessierCogecoMedia
added a commit
to AlexiTessierCogecoMedia/git_bash_windows_powerline
that referenced
this pull request
Feb 19, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
On my system using native bash functions instead of wc -l speeds up the last_status by about 0.2 seconds. This may be because windows pipes are slow or some issue with wc.
Before:
After:
Performance numbers aren't exact as they vary a bit, but you can see a bit difference in the runtime for last_status.