[FFmpeg-devel] [PATCH 1/5] lavc: add a table of all codecs names

Michael Niedermayer michaelni at gmx.at
Sat Aug 20 00:41:56 CEST 2011


On Wed, Aug 17, 2011 at 07:32:03PM +0200, Nicolas George wrote:
> The attached patch does as the commit message says.
> 
> The next patches change a few places to use it and either simplify the code
> or make the output more user friendly.
> 
> The code, as it is, try to uses the field name of the main decoder or
> encoder. It reduces the footprint of about 1400 bytes.
> 
> I hesitated about the failback case when no name could be found. In the end,
> I chose to go for the simplest, lightest API, which makes it really
> appealing to just put avcodec_get_name whenever needed, without requiring a
> buffer.
> 
> Regards,
> 
> -- 
>   Nicolas George

>  .gitignore                |    1 
>  libavcodec/Makefile       |    6 +++
>  libavcodec/avcodec.h      |    6 +++
>  libavcodec/codec_names.sh |   85 ++++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/utils.c        |   19 ++++++++++
>  5 files changed, 117 insertions(+)
> 91405565caa4fb586c0ff20c8538ef4190cc4024  0001-lavc-add-a-table-of-all-codecs-names.patch
> From d50c3b4ac30ddd2fdf4c28e517f265b84eab458b Mon Sep 17 00:00:00 2001
> From: Nicolas George <nicolas.george at normalesup.org>
> Date: Wed, 17 Aug 2011 13:54:49 +0200
> Subject: [PATCH 1/5] lavc: add a table of all codecs names.

i still think this is overkill but if you want to maintain this,
it LGTM and thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Incandescent light bulbs waste a lot of energy as heat so the EU forbids them.
Their replacement, compact fluorescent lamps, much more expensive, dont fit in
many old lamps, flicker, contain toxic mercury, produce a fraction of the light
that is claimed and in a unnatural spectrum rendering colors different than
in natural light. Ah and we now need to turn the heaters up more in winter to
compensate the lower wasted heat. Who wins? Not the environment, thats for sure
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110820/b083b5f5/attachment.asc>


More information about the ffmpeg-devel mailing list