[FFmpeg-devel] [PATCH] pkt_pts reordering

[Michael Niedermayer]

On Fri, Jan 07, 2011 at 08:18:05PM +0100, Vladimir Pantelic wrote:
> Michael Niedermayer wrote:
>
>> --- a/libavcodec/avcodec.h
>> +++ b/libavcodec/avcodec.h
>> @@ -998,6 +998,7 @@ typedef struct AVPanScan{
>>        * that time,\
>>        * the decoder reorders values as needed and sets AVFrame.reordered_opaque\
>>        * to exactly one of the values provided by the user through AVCodecContext.reordered_opaque \
>> +     * @deprecated in favor of pkt_pts\
>>        * - encoding: unused\
>>        * - decoding: Read by user.\
>>        */\
>> @@ -2579,6 +2580,7 @@ typedef struct AVCodecContext {
>>       /**
>>        * opaque 64bit number (generally a PTS) that will be reordered and
>>        * output in AVFrame.reordered_opaque
>> +     * @deprecated in favor of pkt_pts
>>        * - encoding: unused
>>        * - decoding: Set by user.
>>        */
>
> Shouldn't we keep a method for the user to attach an arbitrary
> *opaque* value to a decoded frame?

i can drop the deprecation patch from the set if people want?

note though its just marking it deprecated its not put under ifdefs for actual
removial.

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

I hate to see young programmers poisoned by the kind of thinking
Ulrich Drepper puts forward since it is simply too narrow -- Roman Shaposhnik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110107/dd98c2ff/attachment.pgp>