[FFmpeg-devel] [PATCH] RDT/Realmedia patches #2

[Ronald S. Bultje]

Hi,

On Mon, Nov 17, 2008 at 5:11 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Mon, Nov 17, 2008 at 09:40:34AM -0500, Ronald S. Bultje wrote:
>> @@ -324,11 +325,12 @@
>>      if (!s->parse_packet)
>>          return -1;
>>
>> -    if (!buf) {
>> +    if (!buf && s->prev_stream_id != -1) {
>>          /* return the next packets, if any */
>>          timestamp= 0; ///< Should not be used if buf is NULL, but should be set to the timestamp of the packet returned....
>>          rv= s->parse_packet(s->dynamic_protocol_context,
>> -                            s->st, pkt, &timestamp, NULL, 0, flags);
>
>> +                            s->ic->streams[s->st->index + s->prev_stream_id],
>
> s->st has a poor name and lacks documentation

Attached patch adds docs for s->ic and s->st. The name is consistent
with RTPDemuxContext, which I like and would like to keep this way,
with the goal of eventually merging parts of the two so they can be
commonly accessed without having to care about the type of context
we're dealing with... However, as always, if it's important to you,
I'll change it, let me know.

Ronald
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rdt-document-ic-and-st.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081117/617cea32/attachment.asc>