[FFmpeg-devel] [PATCH 2/2] dshow: handle events in graph

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Dec 9 18:56:22 CET 2012


On Sun, Dec 09, 2012 at 12:45:05PM -0500, Don Moir wrote:
> Ok some people are trying to remove braces and some keep adding
> them... I hope its not a brain freeze if you have to add them later
> at some separate time :) For me personally it just adds to the code
> clutter which is already prevalent.

We prioritize reducing clutter in patches over minimal clutter
in code.
The result of that is to add {} whenever that does not require
extra lines, since it has the potential of improving readability
of future patches significantly (when someone adds a new {
you'd otherwise have to carefully check whether or not they
also changed the if condition).


More information about the ffmpeg-devel mailing list