[FFmpeg-devel] [PATCH] oops I broke rdt.c

Ronald S. Bultje rsbultje
Sun Dec 28 01:26:03 CET 2008


Hi,

2008/12/16 Ronald S. Bultje <rsbultje at gmail.com>:
> $subj says it all, my recent commit to remove usage of RMContext in
> rdt (r16110) broke RDT reading, not sure how that could happen but
> it's probably poor testing (i.e. testing with streams that don't
> actually use that function) and focussing my testing more on plain RM
> files (that patch is part of the set of patches to implement RMStream
> etc.) and forgetting to do thorough testing on RDT streams.
>
> Attached are three patches:
> - one to actually use the index in an array instead of the array itself;
> - one to update audio_pkt_cnt after calls to the ff_rm_retrieve_cache();
> - one to reindent after #2.

So, as a followup, here's three new patches for the above. I realized
I don't actually need the index, so attached patches:
- #1 removes them
- #2 fixes ff_rm_retrieve_cache()
- #3 reindents after #1+#2

Thanks,
Ronald
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: x
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081227/7d5aaf7a/attachment.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: x2
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081227/7d5aaf7a/attachment.asc>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: post-audio_pkt_cnt-reindent.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081227/7d5aaf7a/attachment-0001.txt>



More information about the ffmpeg-devel mailing list