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