Clean git macro functions

Clean up functions that simply run some git command and print its output
or return its exit code.

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

1 files changed, 3 insertions, 9 deletions

diff --git a/git-sonar b/git-sonar
index ade12bf..b2d140e 100755
--- a/git-sonar
+++ b/git-sonar
@@ -119,17 +119,11 @@ fetch() {
 }
 
 commit_short_sha() {
-  printf '%s' "$(git rev-parse --short HEAD 2>/dev/null)"
+  git rev-parse --short HEAD 2>/dev/null
 }
 
 branch_name() {
-  name="$(git symbolic-ref --short HEAD 2>/dev/null)"
-  retcode="$?"
-  if [[ "$retcode" == "0" ]]; then
-    printf %s "$name"
-  else
-    return 1
-  fi
+  git symbolic-ref --short HEAD 2>/dev/null
 }
 
 remote_branch_name() {
@@ -205,7 +199,7 @@ remote_ahead_of_master() {
 #both="\xE2\x83\xA1%{$reset_color%}"
 
 porcelain_status() {
-  printf '%s' "$(git status --porcelain 2>/dev/null)"
+  git status --porcelain 2>/dev/null
 }
 
 staged_status() {