]> git.ipfire.org Git - thirdparty/systemd.git/blobdiff - CODING_STYLE
networkd: disable IPv6 for bridge slave
[thirdparty/systemd.git] / CODING_STYLE
index e5ba396368d8f49f3a77e62d82246fa89f26cde1..b689355c9a6611fdadfe09ab88f15ff3e61930a5 100644 (file)
   b) socket() and socketpair() must get SOCK_CLOEXEC passed
   c) recvmsg() must get MSG_CMSG_CLOEXEC set
   d) F_DUPFD_CLOEXEC should be used instead of F_DUPFD, and so on
+  f) invocations of fopen() should take "e"
 
 - We never use the POSIX version of basename() (which glibc defines it in
   libgen.h), only the GNU version (which glibc defines in string.h).
 
       unlink("/foo/bar/baz");
 
+  Don't cast function calls to (void) that return no error
+  conditions. Specifically, the various xyz_unref() calls that return a NULL
+  object shouldn't be cast to (void), since not using the return value does not
+  hide any errors.
+
 - Don't invoke exit(), ever. It is not replacement for proper error
   handling. Please escalate errors up your call chain, and use normal
   "return" to exit from the main function of a process. If you