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

[Clément Bœsch]

On Mon, Aug 17, 2015 at 03:31:16PM +0200, Michael Niedermayer wrote:
> 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
> 

i actually have a WIP on that (to integrate the visualization of qp in
codecview), which i'll try to submit asap

> your patch adds qp/mb_type stuff under the #ifs
> 
> 


-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150817/d9cea1a5/attachment.sig>