[FFmpeg-cvslog] doc/developer: Better {} style rule

Michael Niedermayer git at videolan.org
Sat Mar 1 04:19:44 EET 2025


ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Thu Feb 27 02:02:12 2025 +0100| [469b7a0ee444f208e8cc0570a9b9ba761d2f04c2] | committer: Michael Niedermayer

doc/developer: Better {} style rule

This makes developer.texi consistent with tools/patcheck

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=469b7a0ee444f208e8cc0570a9b9ba761d2f04c2
---

 doc/developer.texi | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/doc/developer.texi b/doc/developer.texi
index a1bfe180c9..757e0f0ac1 100644
--- a/doc/developer.texi
+++ b/doc/developer.texi
@@ -179,18 +179,17 @@ int fields = ilace ? 2 : 1;
 @end example
 
 @item
-No braces around single-line blocks:
+Don't wrap single-line blocks in braces. Use braces only if there is an accompanying else statement. This keeps future code changes easier to keep track of.
 
 @example c, good
 // Good
-if (bits_pixel == 24)
+if (bits_pixel == 24) @{
     avctx->pix_fmt = AV_PIX_FMT_BGR24;
-else if (bits_pixel == 8)
+@} else if (bits_pixel == 8) @{
     avctx->pix_fmt = AV_PIX_FMT_GRAY8;
-else @{
-    av_log(avctx, AV_LOG_ERROR, "Invalid pixel format.\n");
+@} else
     return AVERROR_INVALIDDATA;
-@}
+
 @end example
 
 @item



More information about the ffmpeg-cvslog mailing list