[FFmpeg-devel] [PATCH] pkt_pts reordering

[Michael Niedermayer]

On Mon, Jan 10, 2011 at 10:10:54AM +0100, Vladimir Pantelic wrote:
> Michael Niedermayer wrote:
>> 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?
>
> see attached

whats the use case for non timestamp opaque?

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

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- 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/20110110/7a85a31e/attachment.pgp>