stonewareslord/lpr
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix redundant (and slow): echo $(_lp_escape ...)

Browse Source 
_lp_escape is: print "%q" "$*"
So echo $(_lp_escape ...) is a slow synonym for _lp_escape

Not all the usage cases are fixed in this commit. The remaining ones are
fixed in other patches with more global optimizations.
This commit is contained in:
Olivier Mengué 2013-05-16 19:41:25 +02:00
parent fe6d9f3ebe
commit 46dbc6157f
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
liquidprompt
View File

@ -777,7 +777,7 @@ _lp_hg_branch()
[[ "$LP_ENABLE_HG" != 1 ]] && return
local branch
branch="$(hg branch 2>/dev/null)"
[[ $? -eq 0 ]] && echo $(_lp_escape "$branch")
[[ $? -eq 0 ]] && _lp_escape "$branch"
}
@ -887,7 +887,7 @@ _lp_fossil_branch()
local branch
branch=$(fossil status 2>/dev/null | grep tags: | cut -c17-)
if [[ -n "$branch" ]] ; then
echo $(_lp_escape "$branch")
_lp_escape "$branch"
else
if fossil info &>/dev/null ; then
echo "no-tag"
@ -956,7 +956,7 @@ _lp_fossil_branch_color()
branch="${LP_COLOR_CHANGES}$branch${NO_COL}$ret${LP_COLOR_CHANGES}$C2A${NO_COL}"
fi
fi
echo -ne $branch # $(_lp_escape "$branch")
_lp_escape "$branch"
fi
}
@ -969,7 +969,7 @@ _lp_bzr_branch()
local branch
branch=$(bzr nick 2> /dev/null)
[[ $? -ne 0 ]] && return
echo $(_lp_escape "$branch")
_lp_escape "$branch"
}