[FFmpeg-cvslog] r12759 - in trunk/libavcodec: aac_ac3_parser.c aac_ac3_parser.h aac_parser.c ac3_parser.c

[Bartlomiej Wolowiec]

On ?roda, 9 kwietnia 2008, Michael Niedermayer wrote:
> I suspect that the best course of action is to revert the changes up to the
> point where the regression tests pass again.
> Next the rm demuxer must be fixed which will likely change the regression
> checksums, which need to be updated after confirming that the rm demuxer
> works equally well or better.
> And last the parser should be commited again. (but of course only if
> regressions pass, if they dont the bugs causing the failure must be found
> and fixed similar to the rm demxuer _before_ commiting the parser)
>
> [...]

This is patch version with all corrections. But if after ac3 frames are eac3 
frames, I don't know what to do to have seektest passed?

tests/data/a-ac3.rm
[...]
-ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:271 size:556 flags:1
+ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:839 size:556 flags:1
 ret: 0 st:-1 ts:2.576668 flags:0
-ret: 0 st: 0 dts:2.960000 pts:2.960000 pos:48659 size:558 flags:1
+ret: 0 st: 0 dts:2.960000 pts:2.960000 pos:49229 size:558 flags:1
[...]

tests/data/b-libav.rm:
[...]
-ret: 0 st: 1 dts:0.975000 pts:0.975000 pos:355101 size:278 flags:1
+ret: 0 st: 1 dts:0.975000 pts:0.975000 pos:-1 size:278 flags:1
 ret: 0 st: 1 ts:1.471000 flags:1
-ret: 0 st: 1 dts:0.975000 pts:0.975000 pos:355101 size:278 flags:1
+ret: 0 st: 1 dts:0.975000 pts:0.975000 pos:-1 size:278 flags:1
[...]

-- 
Bartlomiej Wolowiec
-------------- next part --------------
A non-text attachment was scrubbed...
Name: parser.patch
Type: text/x-diff
Size: 9074 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20080412/fc3fd0e9/attachment.patch>