chiark / gitweb /
CODING_STYLE: don't clobber arguments on failure
authorLennart Poettering <lennart@poettering.net>
Mon, 27 Oct 2014 17:09:07 +0000 (18:09 +0100)
committerLennart Poettering <lennart@poettering.net>
Mon, 27 Oct 2014 17:09:26 +0000 (18:09 +0100)
CODING_STYLE

index 598d241..4439ee6 100644 (file)
   is_main_thread() to detect whether the calling thread is the main
   thread.
 
-- Option parsing:
+- Command line option parsing:
   - Do not print full help() on error, be specific about the error.
   - Do not print messages to stdout on error.
   - Do not POSIX_ME_HARDER unless necessary, i.e. avoid "+" in option string.
+
+- Do not write functions that clobber call-by-reference variables on
+  failure. Use temporary variables for these cases and change the
+  passed in variables only on success.