From b93e69830a5606712f1ec0132e250839afbd87c4 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Wed, 3 Nov 2010 17:07:33 -0700 Subject: [PATCH] CodingStyle: Mention that "free" functions should accept a null pointer. I've always written OVS code this way, so this just codifies it so that others know. --- CodingStyle | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/CodingStyle b/CodingStyle index bbb492e3..2ee189fa 100644 --- a/CodingStyle +++ b/CodingStyle @@ -156,6 +156,12 @@ parameters and their corresponding size parameters should be paired. ... } +Functions that destroy an instance of a dynamically-allocated type +should accept and ignore a null pointer argument. Code that calls +such a function (including the C standard library function free()) +should omit a null-pointer check. We find that this usually makes +code easier to read. + FUNCTION PROTOTYPES -- 2.30.2