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

Ronald S. Bultje rsbultje
Thu Oct 2 14:14:00 CEST 2008


Hi Luca,

On Thu, Oct 2, 2008 at 2:12 AM, Luca Abeni <lucabe72 at email.it> wrote:
> Ronald S. Bultje wrote:
>> You'll notice
>> that I'm not making RTPDemuxContext opaque here, but rather the access
>> to it. void* here isn't actually a RTPDemuxContext anymore after this
>> patch, but rather a datatype context (e.g. h264 or mpeg; see how the
>> caller changes from passing "s" to "s->dynamic_protocol_context"). I
>> propose the name RTPPacketDataContext, is that OK?
>
> I do not like the "RTP" part of the name. "PacketDataContext" is ok, I
> thin. Or "PayloadContext" (since the structure contains data needed to
> parse the payload, right?)

PayloadContext it is then. Patch will hopefully come this weekend.

Ronald




More information about the ffmpeg-devel mailing list