[FFmpeg-devel] [libav-devel] [PATCH] Change behviour of empty AVFilterFormats lists

[Michael Niedermayer]

On Wed, Jun 08, 2011 at 01:55:53AM +0200, Stefano Sabatini wrote:
> On date Tuesday 2011-06-07 17:51:27 +0300, Mina Nagy Zaki encoded:
[...]
> > From 51d8dea811f2a267050d0ffa9e6521b642da363e Mon Sep 17 00:00:00 2001
> > From: Mina Nagy Zaki <mnzaki at gmail.com>
> > Date: Tue, 7 Jun 2011 17:43:30 +0300
> > Subject: [PATCH 2/2] lavfi: Make an empty AVFilterFormats mean 'all formats'
> >  supported.
> > 
> > This makes avfilter_all_formats unnecessary, and so it was removed.
> 
> I have to confess that I don't like this change. Indeed to me "empty
> formats list" has a different meaning than "list containing all
> supported formats", and at least for me it's making harder to
> understand the code.

seconded


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- 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/20110608/e6142199/attachment.asc>