[FFmpeg-cvslog] r17306 - trunk/libavcodec/xvmc.h

[Michael Niedermayer]

On Sun, Feb 15, 2009 at 02:35:32AM +0100, iive wrote:
> Author: iive
> Date: Sun Feb 15 02:35:32 2009
> New Revision: 17306
> 
> Log:
> Detailed description of the interesting fields in xvmc struct.
> 
> Modified:
>    trunk/libavcodec/xvmc.h
> 
> Modified: trunk/libavcodec/xvmc.h
> ==============================================================================
> --- trunk/libavcodec/xvmc.h	Sun Feb 15 02:28:50 2009	(r17305)
> +++ trunk/libavcodec/xvmc.h	Sun Feb 15 02:35:32 2009	(r17306)
> @@ -58,17 +58,35 @@ struct xvmc_render_state {
>  #endif
>  //}@
>  
> -/** modified by calling application and the decoder */
> -//@{
> -    int             start_mv_blocks_num;          ///< offset in the array for the current slice, updated by vo
> -    int             filled_mv_blocks_num;         ///< processed mv block in this slice, changed by decoder

> +    /** Offset in the mv array for the current slice:

this is incomplete, i mean its a offset into .. but what does this offset
mean/represent?


> +        - application - zeros it on  get_buffer().
> +                        successful draw_horiz_band() may increment it
> +                        with filled_mb_block_num or zero both.
> +        - libavcodec  - unchanged
> +    */
> +    int             start_mv_blocks_num;
> +

> +    /** Processed mv blocks in this slice:

here you say processed, and the variable says filled?
also processed/filled by the decoder or application?

anyway +2 from 10 for clarifying app vs lavc for less than a 1/4 of the
vars

[...]


>  #if LIBAVCODEC_VERSION_MAJOR < 53
>  //@{
> -    int             state;                        ///< 0 - free, 1 - waiting to display, 2 - waiting for prediction
> +    /** State - used to workaround limitations in MPlayer vo system.
> +        0   -Surface not used
> +        1   -Surface is still hold in application to be displayed or is still visible.
> +        2   -Surface is still hold in lavcodec buffer for prediction
> +    */
> +    int             state;

ok, you get the +3 points for clarifying state, just one little question
0..2 are the only allowed values? so 3 is not, iam asking because
ive seen |= 2 or so in vo_xvmc.c yesterday and that should be =2 in that
case

diego 24
ivan 14.5

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

He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20090215/8ab78b2f/attachment.pgp>