[FFmpeg-devel] [PATCH] avcodec/libvpx: set CODEC_CAP_EXPERIMENTAL correctly instead of testing at codec open time

[Michael Niedermayer]

On Mon, Dec 09, 2013 at 06:35:17PM -0300, James Almer wrote:
> On 09/12/13 6:08 PM, James Zern wrote:
> > On Sun, Dec 8, 2013 at 2:14 PM, James Almer <jamrial at gmail.com> wrote:
> >> On 08/12/13 10:19 AM, Michael Niedermayer wrote:
> >>> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> >>> ---
> >>>  libavcodec/libvpx.c    |   13 ++++---------
> >>>  libavcodec/libvpx.h    |    2 +-
> >>>  libavcodec/libvpxdec.c |    4 +---
> >>>  libavcodec/libvpxenc.c |    4 +---
> >>>  4 files changed, 7 insertions(+), 16 deletions(-)
> >>>
> >>
> >> Maybe we should leave a log message about libvpx < 1.3.0 as an
> >> AV_LOG_WARNING line.
> >> The idea is that the warning will show up after running ffmpeg with
> >> -strict -2, so the user knows that what's experimental is VP9 encoding
> >> with their libvpx version and not ffmpeg's wrapper, and that they
> >> should consider updating it.
> > 
> > This now happens at registration time, so the log would be issued even
> > if you weren't using vp9.
> 
> The log can for example be in vp9_init() (Both encoder and decoder). That 
> way it will only trigger for VP9 and libvpx < 1.3.0.
> It will however need an extra version check there.

i can add such check if wanted ?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- 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/20131210/7cfea764/attachment.asc>