Skip to content

Commit 0847d9e

Browse files
Improve generator.sh syntax with ShellCheck v0.4.6
1 parent a8bdf8e commit 0847d9e

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

β€Žgenerator.sh

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ if [ ! -r "${distFolder}/${distPrefix}.bash" ] || [ ! -s "${distFolder}/${distPr
1414
alias echo.BoldYellow='echo'
1515
else
1616
# use ColorEcho
17+
# shellcheck source=dist/ColorEcho.bash
1718
. "${distFolder}/${distPrefix}.bash"
1819
command -v echo.BoldRed &> /dev/null || alias echo.BoldRed='echo'
1920
command -v echo.BoldGreen &> /dev/null || alias echo.BoldGreen='echo'
@@ -42,7 +43,7 @@ do
4243
endSym='}'
4344
endIf='fi'
4445
brackets=
45-
para='@'
46+
para='*'
4647
;;
4748
"ksh")
4849
fn='function '
@@ -52,7 +53,7 @@ do
5253
endSym='}'
5354
endIf='fi'
5455
brackets=
55-
para='@'
56+
para='*'
5657
;;
5758
"fish")
5859
fn='function '
@@ -67,12 +68,13 @@ do
6768
"sh")
6869
fn=
6970
dot=
71+
# shellcheck disable=SC2016
7072
echo='$ECHO'
7173
startSym='{'
7274
endSym='}'
7375
endIf='fi'
7476
brackets='()'
75-
para='@'
77+
para='*'
7678
esac
7779

7880
newDist="${distFolder}/${distPrefix}.${shell}"
@@ -92,7 +94,7 @@ else
9294
fi
9395
SH_ECHO
9496
fi
95-
for color in $(awk '{print $1}' "$table")
97+
awk '{print $1}' "$table" | while IFS= read -r color
9698
do
9799
#light or not
98100
for light in "" "Light"
@@ -123,7 +125,7 @@ SH_ECHO
123125
fi
124126
#write the code down
125127
echo "$startSym"
126-
echo " $echo"' -e "\033['"${ulCode}${bCode}${code}"$(grep $color "$table" | awk '{print $2}')'m$'$para'\033[m"'
128+
echo " $echo"' -e "\033['"${ulCode}${bCode}${code}""$(grep "$color" "$table" | awk '{print $2}')"'m$'"$para"'\033[m"'
127129
echo "$endSym"
128130
} >> "$newDist"
129131
done
@@ -135,13 +137,13 @@ SH_ECHO
135137
fnName="${fn} echo${dot}Rainbow${brackets}"
136138
case "$shell" in
137139
"fish")
138-
ifCond="if type lolcat > /dev/null"
140+
ifCond="if which lolcat > /dev/null"
139141
;;
140142
"ksh")
141-
ifCond='if type lolcat 2> /dev/null >&2; then'
143+
ifCond='if which lolcat 2> /dev/null >&2; then'
142144
;;
143145
*)
144-
ifCond='if type lolcat > /dev/null 2>&1; then'
146+
ifCond='if which lolcat > /dev/null 2>&1; then'
145147
;;
146148
esac
147149

0 commit comments

Comments
Β (0)