[SCM] Debian Live manual branch, master, updated. 1a0d278c0459c5f6d3133dffda47a93754f68670

Daniel Baumann daniel at debian.org
Sat Jan 10 21:12:09 UTC 2009


The following commit has been merged in the master branch:
commit 1a0d278c0459c5f6d3133dffda47a93754f68670
Author: Daniel Baumann <daniel at debian.org>
Date:   Sat Jan 10 22:11:58 2009 +0100

    Adding item in coding style about variable assignements.

diff --git a/xml/chapters/coding-style.xml b/xml/chapters/coding-style.xml
index ee072fa..d4fd2dc 100644
--- a/xml/chapters/coding-style.xml
+++ b/xml/chapters/coding-style.xml
@@ -88,6 +88,21 @@ foo ()
 	<listitem>Local variables start with __LH_ prefix.</listitem>
 	<listitem>Put curly brackets arround them, write ${FOO} not $FOO.</listitem>
 	<listitem>Always protect variables wrt/ potential whitespaces, write "${FOO}" not ${FOO}.</listitem>
+	<listitem>For consistency reasons, always use quotes when assigning values to variables:
+
+	Bad:
+
+<screen>
+FOO=bar
+</screen>
+
+	Good:
+
+<screen>
+FOO="bar"
+</screen>
+	</listitem>
+
 	<listitem>If multiple variables are used, quote the full expression:
 
 	Bad:

-- 
Debian Live manual



More information about the debian-live-changes mailing list