[FFmpeg-devel] [PATCH] nobody cares: use designated initializers

Clément Bœsch ubitux at gmail.com
Fri Dec 30 13:18:50 CET 2011


On Fri, Dec 30, 2011 at 12:02:05PM +0000, Paul B Mahol wrote:

I do care ;)

> ---
>  libavcodec/bintext.c          |   51 +++++++++++++++++++---------------------
>  libavcodec/diracdec.c         |    1 -
>  libavcodec/g729dec.c          |   14 +++++------
>  libavcodec/j2kdec.c           |   15 +++++------
>  libavcodec/j2kenc.c           |   14 +++++-----
>  libavcodec/libaacplus.c       |   14 +++++-----
>  libavcodec/libopenjpegenc.c   |    1 -
>  libavcodec/libutvideo.cpp     |   22 ++++++-----------

Do not do it for C++ please. Maybe add a note about this (you're not the
first one to propose such change).

Maybe also mention in the commit description you removed some pointless
NULL assignments.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111230/ee2ea952/attachment.asc>


More information about the ffmpeg-devel mailing list