From 800b1b1e0aa375505b02eb0e71f6e7709f898df1 Mon Sep 17 00:00:00 2001
From: Jeff Byrnes <thejeffbyrnes@gmail.com>
Date: Wed, 29 Jul 2020 17:48:53 -0400
Subject: [PATCH] Avoid running ll code if its not used

Prior to this, if $shell_ls_aliases_enabled="false", this code would
still run, but unnecessarily.
---
 lib/git/shell_shortcuts.sh | 32 ++++++++++++++++----------------
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/lib/git/shell_shortcuts.sh b/lib/git/shell_shortcuts.sh
index 54f79ce0..43b72156 100644
--- a/lib/git/shell_shortcuts.sh
+++ b/lib/git/shell_shortcuts.sh
@@ -92,25 +92,25 @@ if [ "$shell_command_wrapping_enabled" = "true" ] || [ "$bash_command_wrapping_e
 fi
 
 
-# BSD ls is different to Linux (GNU) ls
-# Test for BSD ls
-if ! ls --color=auto > /dev/null 2>&1; then
-  # ls is BSD
-  _ls_bsd="BSD"
-fi
-
-# Test if readlink supports -f option, test for greadlink on Mac, then fallback to perl
-if \readlink -f / > /dev/null 2>&1; then
-  _abs_path_command=(readlink -f)
-elif greadlink -f / > /dev/null 2>&1; then
-  _abs_path_command=(greadlink -f)
-else
-  _abs_path_command=(perl -e 'use Cwd abs_path; print abs_path(shift)')
-fi
-
 # Function wrapper around 'll'
 # Adds numbered shortcuts to output of ls -l, just like 'git status'
 if [ "$shell_ls_aliases_enabled" = "true" ] && builtin command -v ruby > /dev/null 2>&1; then
+  # BSD ls is different to Linux (GNU) ls
+  # Test for BSD ls
+  if ! ls --color=auto > /dev/null 2>&1; then
+    # ls is BSD
+    _ls_bsd="BSD"
+  fi
+
+  # Test if readlink supports -f option, test for greadlink on Mac, then fallback to perl
+  if \readlink -f / > /dev/null 2>&1; then
+    _abs_path_command=(readlink -f)
+  elif greadlink -f / > /dev/null 2>&1; then
+    _abs_path_command=(greadlink -f)
+  else
+    _abs_path_command=(perl -e 'use Cwd abs_path; print abs_path(shift)')
+  fi
+
   unalias ll > /dev/null 2>&1; unset -f ll > /dev/null 2>&1
   function ls_with_file_shortcuts {
     local ll_output