[Ffmpeg-devel] doxygen compatible @deprecated tag

Michael Niedermayer michaelni
Mon Apr 30 14:39:28 CEST 2007


Hi

On Mon, Apr 30, 2007 at 05:43:03PM +1000, mark cox wrote:
> On 4/30/07, Guillaume POIRIER <poirierg at gmail.com> wrote:
> >
> >Hi,
> >
> >On 4/30/07, mark cox <melbournemark+ffmpeg-devel at gmail.com> wrote:
> >>
> >> New patch with comments applied.

no it is not applied, not even attached it was ...


> >> mark
> >
> >You forgot to attach it apparently...
> 
> 
> Attached.
> 
> Guillaume

your name is not Guillaume



> >
> >--
> >Rich, you're forgetting one thing here: *everybody* except you is
> >stupid.
> >    M?ns Rullg?rd
> >_______________________________________________
> >ffmpeg-devel mailing list
> >ffmpeg-devel at mplayerhq.hu
> >http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel

may i suggest that you snip irrelevant parts of the message to which you
reply


> Index: libavcodec/eval.c
> ===================================================================
> --- libavcodec/eval.c	(revision 8858)
> +++ libavcodec/eval.c	(working copy)
> @@ -427,6 +427,9 @@
>  }
> 
>  #if LIBAVCODEC_VERSION_INT < ((52<<16)+(0<<8)+0)
> +/*
> + * @deprecated Use ff_eval2 instead
> +*/

/**

* is not aligned

and comments belong into the headers if headers exist


>  attribute_deprecated double ff_eval(char *s, double *const_value, const char **const_name,
>                 double (**func1)(void *, double), const char **func1_name,
>                 double (**func2)(void *, double, double), char **func2_name,
> Index: libavcodec/avcodec.h
> ===================================================================
> --- libavcodec/avcodec.h	(revision 8858)
> +++ libavcodec/avcodec.h	(working copy)
> @@ -2149,7 +2149,9 @@
>   * AVPaletteControl
>   * This structure defines a method for communicating palette changes
>   * between and demuxer and a decoder.
> - * This is totally broken, palette changes should be sent as AVPackets.
> + *
> + * @deprecated Use AvPackets to send palette changes instead.
> + * This is totally broken.

AVPacket


>   */
>  #define AVPALETTE_SIZE 1024
>  #define AVPALETTE_COUNT 256
> @@ -2464,13 +2466,25 @@
>  #if LIBAVCODEC_VERSION_INT < ((52<<16)+(0<<8)+0)
>  /* YUV420 format is assumed ! */
> 
> +/*
> + * @deprecated Use the software scaler (swscale) instead.
> + */

/**


[...]
> @@ -2605,6 +2624,9 @@
> 
>  #if LIBAVCODEC_VERSION_INT < ((52<<16)+(0<<8)+0)
>  /* convert among pixel formats */
> +/*
> + * @deprecated Use the software scaler (swscale) instead.
> + */

merge new and existing comments instead of adding a second one


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

If you really think that XML is the answer, then you definitly missunderstood
the question -- Attila Kinali
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070430/95473786/attachment.pgp>



More information about the ffmpeg-devel mailing list