From 7f8bf08f9036de419ad14c55b61eda74c6659d3a Mon Sep 17 00:00:00 2001
From: Lennart Poettering <lennart@poettering.net>
Date: Mon, 27 Oct 2014 18:09:07 +0100
Subject: [PATCH] CODING_STYLE: don't clobber arguments on failure

---
 CODING_STYLE | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/CODING_STYLE b/CODING_STYLE
index 598d241b2..4439ee609 100644
--- a/CODING_STYLE
+++ b/CODING_STYLE
@@ -182,7 +182,11 @@
   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.
-- 
2.30.2