t0027: Tests for core.eol=native, eol=lf, eol=crlf
[gitweb.git] / Documentation / CodingGuidelines
index 3d08671ad3fee3bbe2a3af7332080145a15dfc0a..4d90c77c7bd643b11df8d3094efe142493df53b9 100644 (file)
@@ -151,6 +151,19 @@ For shell scripts specifically (not exhaustive):
    interface translatable. See "Marking strings for translation" in
    po/README.
 
+ - We do not write our "test" command with "-a" and "-o" and use "&&"
+   or "||" to concatenate multiple "test" commands instead, because
+   the use of "-a/-o" is often error-prone.  E.g.
+
+     test -n "$x" -a "$a" = "$b"
+
+   is buggy and breaks when $x is "=", but
+
+     test -n "$x" && test "$a" = "$b"
+
+   does not have such a problem.
+
+
 For C programs:
 
  - We use tabs to indent, and interpret tabs as taking up to