Module: libav
Branch: master
Commit: e004bc16a1304822226b5b1ceebdb899d72ee538
Author: Luca Barbato <lu_z...@gentoo.org>
Committer: Luca Barbato <lu_z...@gentoo.org>
Date: Fri May 4 10:03:42 2012 -0700
doc: clarify check for NULL pointer style
Our code should be terse and clear.
---
doc/developer.texi | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/doc/developer.texi b/doc/developer.texi
index de64239..fed28cd 100644
--- a/doc/developer.texi
+++ b/doc/developer.texi
@@ -73,6 +73,10 @@ const char *avfilter_configuration(void)
@}
@end example
@item
+Do not check for NULL values by comparison, @samp{if (p)} and
+@samp{if (!p)} are correct; @samp{if (p == NULL)} and @samp{if (p != NULL)}
+are not.
+@item
In case of a single-statement if, no curly braces are required:
@example
if (!pic || !picref)
_______________________________________________
libav-commits mailing list
libav-commits@libav.org
https://lists.libav.org/mailman/listinfo/libav-commits
