[FFmpeg-devel] [PATCH] doc/examples/muxing: make compatible with C++

Don Moir donmoir at comcast.net
Fri Mar 14 18:32:40 CET 2014

----- Original Message ----- 
From: "Nicolas George" <george at nsup.org>
To: "FFmpeg development discussions and patches" <ffmpeg-devel at ffmpeg.org>
Sent: Friday, March 14, 2014 12:23 PM
Subject: Re: [FFmpeg-devel] [PATCH] doc/examples/muxing: make compatible with C++

>I am not against a pure c++ example, but making the C examples less readable
>and getting them to show bad practice is IMHO a very bad idea.

Nicolas, sorry just hard for me to let this pass and trying to be objective ( hopefully constructive :)

Seems the 'best practice' in ffmpeg is to write the most terse unreadable code you can.

o - Single letter variables (not including indexes like i, j, etc)

o - Hide as much as posible as long as it's tight and small (even though it does not make any difference in bin size). 

o - poorly documented  (internal code worse)  - the API is sometimes guess work

o - some code that weaves around endlessly

o - no use of mixed text and declarations - A C99 feature but maybe that is optional (optional geeze) - makes code more readable

I don't know, but if GSOC is about student learning, why would they choose ffmpeg? Just a thought.

More information about the ffmpeg-devel mailing list