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

Michael Niedermayer michaelni
Sun Dec 28 03:20:22 CET 2008


On Sat, Dec 27, 2008 at 07:26:03PM -0500, Ronald S. Bultje wrote:
> 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

ok


> - #2 fixes ff_rm_retrieve_cache()

ok

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

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081228/adc324b4/attachment.pgp>



More information about the ffmpeg-devel mailing list