exit -> return

profile.conf

@@ -70,26 +70,41 @@ MAKEFLAGS='-j12'
 PKGSOURCES='/share/src/archives'
 
 [aliases]
+# Aliases section is used to set user aliases, it is loaded only for
+# interactive shells.
+# Various ls aliases
 ll='ls -laFh --color=auto'
 la='ls -Ah --color=auto'
 l='ls -CF --color=auto'
 ls='ls --color=auto'
 
+# Add color to grep output
 grep='grep --color=auto'
 egrep='egrep --color=auto'
 fgrep='fgrep --color=auto'
+
+# Quick find alias
 qfind="find . -name "
 
+# Some alias for compiling
+mk='make'
 mkck='make check'
 mkin='make install'
 mkdin='make DESTDIR=$PWD/dest-install install'
+
+# ssh alias with X11 forwarding, without right restriction
 ssh='ssh -Y'
 
+# Resume mode for wget
 wget='wget -c' # resume mode by default
-myip='curl ip.appspot.com'
 
 # Human readable by default
 df='df -H'
 du='du -ch'
 sdu='du -sk ./* | sort -n'
 hdu='du -hs ./* | sort -H'
+
+# Readable dmesg timestamps
+dmesg='dmesg -T'
+
+# End of profile.conf

profile.d/filefct.sh

@@ -299,7 +299,7 @@ rmspc()
             (
                 local lastdir=$f
                 (( verb )) && disp I "Entering directory $(pwd)/$f ..."
-                pushd "$f" >/dev/null || exit 1
+                pushd "$f" >/dev/null || return 1
 
                 if (( substchar_set )); then
                     rmspc ${recurs:+-r} -c "$substchar" ${verb:+-v} ${shell:+-s}
@@ -307,7 +307,7 @@ rmspc()
                     rmspc ${recurs:+-r} ${verb:+-v} ${shell:+-s}
                 fi
 
-                popd >/dev/null || exit 1
+                popd >/dev/null || return 1
                 (( verb )) && disp I "Leaving directory $(pwd)/$lastdir"
             )
         fi

profile.d/info.sh

@@ -39,11 +39,14 @@
 # Usage: ver
 ver()
 {
-    local PARSED=$(getopt -o h --long help -n 'ver' -- "$@")
+    local PARSED
+
+    PARSED=$(getopt -o h --long help -n 'ver' -- "$@")
     if [[ $? -ne 0 ]]; then
         disp E "Invalid options, use \"ver --help\" to display usage."
         return 1
     fi
+
     eval set -- "$PARSED"
     while true; do
         case "$1" in
@@ -72,11 +75,13 @@ export -f ver
 
 
 # ------------------------------------------------------------------------------
-# Display weather of the given city (or default one)
+# Display weather for the given city (or the default one)
 # Usage: meteo [city1 city2 ...]
 meteo()
 {
-    local PARSED=$(getopt -o h --long help -n 'meteo' -- "$@")
+    local PARSED
+
+    PARSED=$(getopt -o h --long help -n 'meteo' -- "$@")
     if [[ $? -ne 0 ]]; then
         disp E "Invalid options, use \"meteo --help\" to display usage."
         return 1
@@ -85,7 +90,9 @@ meteo()
     while true; do
         case "$1" in
             -h|--help)
-                printf "meteo: Fetch weather data.\nUsage: meteo [city1 city2 ...]\n"
+                printf "meteo: Fetch weather data.\n"
+                printf "Usage: meteo [city1 city2 ...]\n"
+                printf "If no city is provided, the default city from configuration will be used.\n"
                 return 0
                 ;;
             --)
@@ -100,12 +107,13 @@ meteo()
     done
 
     local cities=("$@")
+    local city="" encoded=""
     [[ $# -eq 0 ]] && cities=("$DEFAULT_CITY")
 
     for city in "${cities[@]}"; do
         encoded=$(urlencode "$city")
         dwl "https://wttr.in/$encoded" || \
-            disp E "Failed fetching datas for $city."
+            disp E "Failed to fetch weather data for $city."
     done
 }
 export -f meteo
@@ -117,16 +125,20 @@ export -f meteo
 # Usage: showinfo
 showinfo()
 {
-    local PARSED=$(getopt -o h --long help -n 'showinfo' -- "$@")
+    local PARSED
+
+    PARSED=$(getopt -o h --long help -n 'showinfo' -- "$@")
     if [[ $? -ne 0 ]]; then
         disp E "Invalid options, use \"showinfo --help\" to display usage."
         return 1
     fi
     eval set -- "$PARSED"
+
     while true; do
         case "$1" in
             -h|--help)
-                printf "showinfo: Display system information (hostname, kernel, uptime).\nUsage: showinfo\n"
+                printf "showinfo: Display system information (hostname, kernel, uptime and fetch output when available).\n"
+                printf "Usage: showinfo\n"
                 return 0
                 ;;
             --)
@@ -140,20 +152,20 @@ showinfo()
         esac
     done
 
+    local hostname_str
+    local figopt=()
+    hostname_str="$(hostname)"
+
     printf "\n"
     if command -v figlet >/dev/null 2>&1; then
-        if [[ -s /usr/share/figlet/ansi_shadow.flf ]]; then
-            local figopt="-f ansi_shadow"
-        fi
-        if [[ -n $figopt ]]; then
-            figlet -k $figopt $(hostname)
+        [[ -s /usr/share/figlet/ansi_shadow.flf ]] && \
+            figopt=(-f ansi_shadow)
+        figlet -k "${figopt[@]}" "$hostname_str"
     else
-            figlet $(hostname)
+        printf "%s\n" "$hostname_str"
     fi
-    else
-        hostname -f
-    fi
-    echo ""
+
+    printf "\n"
     if command -v neofetch >/dev/null 2>&1; then
         neofetch
     elif command -v fastfetch >/dev/null 2>&1; then
@@ -163,11 +175,11 @@ showinfo()
             if [[ -s /etc/os-release ]]; then
                 # shellcheck disable=SC1091
                 . /etc/os-release
-                printf "$NAME $VERSION\n"
+                printf "%s %s\n" "$NAME" "$VERSION"
             else
                 cat /proc/version
             fi
-            printf "Uptime: $(uptime -p)\n"
+            printf "Uptime: %s\n" "$(uptime -p)"
         )
     fi
 }