qcm

git clone git://bebou.netlib.re/qcm
Log | Files | Refs | README |

commit 5f8c5792c6ff402b7eb7bd7f2d08ca9b66f25b3f
parent e94ffb8cbf1ed357c88b9201ed0b9733c0135fdb
Auteurice: Arthur Pons <arthur.pons@unistra.fr>
Date:   Wed, 11 Dec 2024 15:53:00 +0100

On ajoute des {} aux fonction sans

Je passe qcm dans shellcheck et il aime pas les fonctions sans {} même
si c'est POSIX. Voir ceci dans leur wiki :

========
POSIX allows the body of a function to be any compound command, e.g.
foo() for i; do :; done. Since this usage is rare, ShellCheck
intentionally requires the body to be { ..; } (or ( ..; )):

    foo() {
      for i; do :; done
    }

This additional structure requirement helps improve error messages and
suggestions by not parsing down a path that less advanced users wouldn't
expect.
========

C'est un choix, je m'y conforme pour que shellcheck me soit quand même
utile.

Diffstat:
M
qcm.sh
 | 
14
+++++++++-----

1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/qcm.sh b/qcm.sh
@@ -1,6 +1,7 @@
 #! /bin/sh
 
-usage() <<-. cat
+usage() {
+<<-. cat
 	Faire des sondages simples en parsant les logs d'un serveur web
 
 	Usage : qcm [-h] [-e] [-u url] [-l logs] [-g] [-i identifiant]
@@ -23,8 +24,10 @@ usage() <<-. cat
 	Pour une explication plus détaillée voir le README ou
 	http://arthur.bebou.netlib.re/qcm
 .
+}
 
-show() <<-. cat
+show() {
+<<-. cat
 	url         : $url/$id/réponse
 	par exemple : $url/$id/A
 
@@ -36,6 +39,7 @@ show() <<-. cat
 
 	total   : $tot
 .
+}
 
 gplot() {
 gnuplot -e "
@@ -162,9 +166,9 @@ else
 	id=$(basename "$tmpd" | cut -d'.' -f2)
 fi
 
-alias question:=question; question() question="$*"
-alias type:=type;         type()     type="$*"
-alias options:=options;   options()  options="$*"
+alias question:=question; question() { question="$*"; }
+alias type:=type;         type()     { type="$*"; }
+alias options:=options;   options()  { options="$*"; }
 alias choix:=choix;       choix()    { [ "$*" = "multiple" ] && choix="+" || choix=; }
 alias unique:=unique;     unique()   { [ "$*" = "non" ] && uopt= || uopt="-u"; }