[hg] Cache the result of the in_hg
call within hg_prompt_info
helpers
The helper functions are smart only work if the user is in an hg repository. Unfortunately checking that fact can take a while for large repos, so these redundant checks are taking a while. This diff stores the result of the first call from `hg_prompt_info` in a local variable and just checks that for every other function. This gave me a multi-second speedup locally.
This commit is contained in:
parent
42a47c2dae
commit
5de6253b0c
@ -26,22 +26,24 @@ function in_hg() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
function hg_get_branch_name() {
|
function hg_get_branch_name() {
|
||||||
if [ $(in_hg) ]; then
|
if [ $_INHG ]; then
|
||||||
echo $(hg branch)
|
echo $(hg branch)
|
||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
|
|
||||||
function hg_prompt_info {
|
function hg_prompt_info {
|
||||||
if [ $(in_hg) ]; then
|
_INHG=$(in_hg)
|
||||||
|
if [ $_INHG ]; then
|
||||||
_DISPLAY=$(hg_get_branch_name)
|
_DISPLAY=$(hg_get_branch_name)
|
||||||
echo "$ZSH_PROMPT_BASE_COLOR$ZSH_THEME_HG_PROMPT_PREFIX\
|
echo "$ZSH_PROMPT_BASE_COLOR$ZSH_THEME_HG_PROMPT_PREFIX\
|
||||||
$ZSH_THEME_REPO_NAME_COLOR$_DISPLAY$ZSH_PROMPT_BASE_COLOR$ZSH_PROMPT_BASE_COLOR$(hg_dirty)$ZSH_THEME_HG_PROMPT_SUFFIX$ZSH_PROMPT_BASE_COLOR"
|
$ZSH_THEME_REPO_NAME_COLOR$_DISPLAY$ZSH_PROMPT_BASE_COLOR$ZSH_PROMPT_BASE_COLOR$(hg_dirty)$ZSH_THEME_HG_PROMPT_SUFFIX$ZSH_PROMPT_BASE_COLOR"
|
||||||
unset _DISPLAY
|
unset _DISPLAY
|
||||||
fi
|
fi
|
||||||
|
unset _INHG
|
||||||
}
|
}
|
||||||
|
|
||||||
function hg_dirty_choose {
|
function hg_dirty_choose {
|
||||||
if [ $(in_hg) ]; then
|
if [ $_INHG ]; then
|
||||||
# Allow turning off the 'hg status' call. Useful in large repos where that
|
# Allow turning off the 'hg status' call. Useful in large repos where that
|
||||||
# can take a while. Mirror the functionality of the git plugin by just
|
# can take a while. Mirror the functionality of the git plugin by just
|
||||||
# showing the "clean" prompt in this case.
|
# showing the "clean" prompt in this case.
|
||||||
|
Loading…
Reference in New Issue
Block a user