fix summary line stripping bug

[small-misc-bash] / ll-function

diff --git a/ll-function b/ll-function
index 7a1c62378fb0dd7d592e5f7282b51da8419c6dac..cdc2c811409513398c12abb83a5e43f75804b723 100644 (file)
--- a/ll-function
+++ b/ll-function
@@ -20,6 +20,7 @@
 # better hard link count: number of subdirectories or number of linked files or omitted if 0
 # better human readable size
 # more natural date/time format for my American raised eyes
+# remove first line size summary
 ll() {
     local x y perm line binls sizePadding middle tail size \
            max_hl_digits hardlinks initial_space hardlink_spacing
@@ -35,14 +36,21 @@ ll() {
     #  or blank for no other kind of acl
     #  I don't want to see this generally.
     while read line; do
-       if $first; then
+        # if we did want the first line, it would need to be stripped of non-printing chars:
+       # line=${line#$'\E[00m'}
+        # when no argument is given to ls, the first line is like "total 123M". strip that
+       if $first && [[ ! $@ ]]; then
            first=false
         else
-            # if we did want the first line, it would need to be stripped of non-printing chars:
-           # line=${line#$'\E[00m'}
             lines+=("$line")
             [[ ! ${line:10:1} == " " ]] && aclchar=true
 
+            # we also need to parse the hardlinks on the first pass, because for
+            # example ls could see the highest count as 11, and thus use 3
+            # places for hardlinks, " 10", but then we use 9 or 8 for a more
+            # useful count, and would then use 2 places. So we have to look
+            # through them all because we can't rely on the spacing that ls
+            # decided on.
             y="${line:11}"
             initial_space="${y%%[![:space:]]*}"
             hardlinks="${y#$initial_space}" # remove any initial spaces
@@ -58,7 +66,7 @@ ll() {
                 max_hl_digits=${#hardlinks}
             fi
             
-            hl+=($hardlinks)
+            hl+=("$hardlinks")
             
         fi
     done< <( "$binls" -lAh --color=always "--time-style=+%m-%d     %Y