[libav-devel] [PATCH 4/4] vc1dec.c: Cosmetics

Kostya Shishkov kostya.shishkov at gmail.com
Tue Nov 8 11:18:30 CET 2011


On Tue, Nov 08, 2011 at 03:47:33PM +0600, Mashiat Sarker Shakkhar wrote:
> There are no reason for "if (c_valid) { ... }" to appear before
> "if (b_valid) { ... }".
> ---
>  libavcodec/vc1dec.c |   20 ++++++++++----------
>  1 files changed, 10 insertions(+), 10 deletions(-)

The message is misleading. First, it's not cosmetic (as it changes the code,
not how it looks). Second, I believe the reason is the way the standard
described it.

Patch per se is harmless.


More information about the libav-devel mailing list