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

Ronald S. Bultje rsbultje
Tue Dec 16 15:00:34 CET 2008


Hi,

$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.

My apologies for the breakage.

Ronald
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rdt-fix-idx.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081216/a5538401/attachment.asc>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rmdec-fix-dont_use_rmcontext.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081216/a5538401/attachment.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rmdec-post-fix_dont_use_rmcontext-reindent.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081216/a5538401/attachment-0001.asc>



More information about the ffmpeg-devel mailing list