[MPlayer-dev-eng] [PATCH] Decode economy - 2

[Michael Niedermayer]

Hi

On Fri, Sep 15, 2006 at 10:52:36PM +0500, Sergey Khlutchin wrote:
> Hi!
> 
> This is a second version of the patch.
> I divided it on two parts and have tested the patched libavcodec for backward compatibility.
> New parameters for libavcodec was moved to AVCodecContext structure and mjpeg decoder was optimized.
> 
> >When the size of video is more than the size of its visible part,
> >it is possible to accelerate decoding not making transformations of invisible blocks of the picture.
> >The attached patch counts the picture visible part in view of used filters and location of the MPlayer's window inside of parents.
> >It was necessary to do changes in different parts, probably some places are insufficiently elegant,
> >and I would like to discuss it.
> >This code works for -vo x11 and MJPEG generated by Elphel camera.
> 
> Sergey Khlutchin
> Elphel, Inc
> 

> Index: libavcodec/mjpeg.c

patches for libav* belong to ffmpeg-dev


[...]
> +		px = mb_x*s->h_max;

tabs are forbiden in libav*


[...]
> -//                    dprintf("mb: %d %d processed\n", mb_y, mb_x);

unrelated 

[...]
> Index: libavcodec/avcodec.h
> ===================================================================
> --- libavcodec/avcodec.h	(revision 6260)
> +++ libavcodec/avcodec.h	(working copy)
> @@ -1776,6 +1776,13 @@
>       int lowres;
>  
>      /**
> +     * Partial decoding. Visible rectangle: x0 <= x < x1
> +     * - encoding: unused
> +     * - decoding: set by user, defaults by avcodec_alloc_context, reset by codec if sizes was changed
> +     */
> +     int vis_x0, vis_x1, vis_y0, vis_y1;

breaks binary compatibility and duplicates skip_bottom and skip_top
the new variables should be names skip_right and skip_left and added to the
end of the struct

[mplayer related changes not reviewd]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is