[FFmpeg-devel] [PATCH] Generalize pixel format enum fields to int formats

S.N. Hemanth Meenakshisundaram smeenaks
Wed Jul 21 19:27:18 CEST 2010


> On Wed, Jul 21, 2010 at 02:38:28AM -0700, S.N. Hemanth Meenakshisundaram
> wrote:
>> Hi,
>>
>> Patch updated with the compact notation for intializing num_formats in
>> avfilter_all_formats and type in avfilter_default_query_formats. This
>> works with make lavfitest and make test. However the next patch in the
>> series "lavfi audio framework" fails those tests. More details on that
>> in that thread.
>>
>> ---


libavfilter/avfilter.c      |    6 ++++--
libavfilter/avfilter.h      |   22 ++++++++++++----------
libavfilter/avfiltergraph.c |    4 ++--
libavfilter/defaults.c      |    6 +++++-
libavfilter/formats.c       |   32 +++++++++++++++++---------------
libavfilter/vf_scale.c      |    4 ++--
6 files changed, 41 insertions(+), 32 deletions(-)


Assert added and trailing whitespace removed.

Where should I add the API changes entry? Also for the conditional in
formats.c, I could replace it with positive conditions but there would
then be more than 2 conditions. is that ok?

Regards,
Hemanth
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Generalize-pixel-format-enum-fields-to-int-formats.patch
Type: text/x-patch
Size: 9220 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100721/7066c252/attachment.bin>



More information about the ffmpeg-devel mailing list