[FFmpeg-devel] [PATCH] Make parser not favor packets with pts/dts (and related fixes)

[Ivan Schreter]

Hi,

Michael Niedermayer wrote:
> patch below does make the parser treat packets equal, <put some joke mixing
> human and packet rights here>
>
> [...]
> this should make it easier to feed pos in the parser as well
> [...]
>   

Attached patches add support for storing packet position alongside 
dts/pts in lavc and using it in lavf to determine correct frame position 
(provided cur_pkt.pos is set correctly). They are prerequisite for 
seeking changes, which rely on having AVPacket.pos set correctly.

lavf patch causes regression in seek test (attached as well), since 
positions are corrected. PTS/DTS is the same, so it seems perfectly OK.

BTW, when committing changes, should the regression change go together 
in one commit with the code which causes the regression, or separately?

Regards,

Ivan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: avcodec_framepos.patch
Type: text/x-patch
Size: 4863 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090304/3a5da374/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: avformat_framepos.patch
Type: text/x-patch
Size: 2976 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090304/3a5da374/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: regress_framepos.patch
Type: text/x-patch
Size: 61570 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090304/3a5da374/attachment-0002.bin>