CodingGuidelines: do not ==/!= compare with 0 or '\0' or NULL
Signed-off-by: Junio C Hamano <gitster@pobox.com> Signed-off-by: Eric Sunshine <sunshine@sunshineco.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Junio C Hamano committed
May 8, 2020 at 13:51 UTC
5c7bb0146e1971a00308a667cb0424459262c273
1 file changed
+12
Documentation/CodingGuidelines
+12
@@ -238,6 +238,18 @@ For C programs:
238
while( condition )
239
func (bar+1);
240
241
+ - Do not explicitly compare an integral value with constant 0 or '\0',
242
+ or a pointer value with constant NULL. For instance, to validate that
243
+ counted array <ptr, cnt> is initialized but has no elements, write:
244
+
245
+ if (!ptr || cnt)
246
+ BUG("empty array expected");
247
+
248
+ and not:
249
+
250
+ if (ptr == NULL || cnt != 0);
251
+ BUG("empty array expected");
252
+
253
- We avoid using braces unnecessarily. I.e.
254
255
if (bla) {