Fix a bug which surfaced in commits_ahead/behind_remoteHEADmaster

If a remote branch is set, but doesn't exist (in the 'missingups'
condition), these functions would output blank ("") causing an error
when used in comparisons in color_local_commits().  It's arguable
whether the POSIX reworks caused this bug or simply revealed it, since
the previous code used bash's [[ ]] test syntax which seemed to tolerate
this case.

Restructure control flow so that any "error" in these functions yields a
result of "0".

The ahead/behind_master functions did not have this problem (the
git-rev-list command failure is already handled), but rewrite them to
match the style of the local functions.

Additionally, the git-rev-list commands are updated to use the more
straight-forward A..B syntax, dropping the use of --left-only or
--right-only.  This is more obvious code, and older git clients may
return incorrect results using the previous syntax.

Signed-off-by: Matt Hunter <m@lfurio.us>

0 files changed, 0 insertions, 0 deletions