[FFmpeg-devel] [PATCH] lavc/ffmpeg: put FF_API_DEBUG_MV around remaining uses of FF_DEBUG_VIS_QP/MB_TYPE.

Michael Niedermayer michael at niedermayer.cc
Mon Aug 17 15:31:16 CEST 2015


On Mon, Aug 17, 2015 at 09:13:58AM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Mon, Aug 17, 2015 at 8:44 AM, Michael Niedermayer <michael at niedermayer.cc
> > wrote:
> 
> > On Mon, Aug 17, 2015 at 08:01:56AM -0400, Ronald S. Bultje wrote:
> > > ---
> > >  ffmpeg.c                   | 6 +++++-
> > >  libavcodec/mpegvideo.c     | 4 ++++
> > >  libavcodec/options_table.h | 2 ++
> > >  libavcodec/pthread_frame.c | 2 ++
> > >  4 files changed, 13 insertions(+), 1 deletion(-)
> >
> > please also remove me from maintainers from the affected codecs
> > in case this code gets actually removed or disabled before a
> > documented replacement is in place
> >
> > i use these for maintaining and testing the code
> 
> 
> (I expected this sort of reply,) so why is it under a deprecation macro?
> 
> Why hasn't anyone taken the time to write a filter that places these things
> on top of a frame, so it could be shared with other codecs that export the
> same information?

ubitux has written one for the motion vectors (codecview)
so these could probably be removed but the qp/mb_type vissualization
still lacks a replacement

your patch adds qp/mb_type stuff under the #ifs


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

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150817/89b84787/attachment.sig>


More information about the ffmpeg-devel mailing list