[FFmpeg-devel] Realmedia patch

[Ronald S. Bultje]

Hi Luca,

On Tue, Sep 2, 2008 at 9:56 AM, Luca Abeni <lucabe72 at email.it> wrote:
> Ronald S. Bultje wrote:
>> On Sun, Aug 31, 2008 at 6:38 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>>> On Sun, Aug 31, 2008 at 3:15 PM, Luca Abeni <lucabe72 at email.it> wrote:
>>>> Ronald S. Bultje wrote:
>>>> [...]
>>>>> +static void *
>>>>> +rdt_new_extradata (void)
>>>>> +{
>>>>> +    rdt_data *rdt = av_mallocz(sizeof(rdt_data) + FF_INPUT_BUFFER_PADDING_SIZE);
>>>> why "+ FF_INPUT_BUFFER_PADDING_SIZE" here?
>>> As discussed, attached is a pach without.
>>
[..]
> I think the patch looks ok (but I cannot comment on the interaction
> with the RM demuxer); I do not know if the
> +extern RTPDynamicProtocolHandler ff_rdt_video_handler, ff_rdt_audio_handler,
> +    ff_rdt_live_video_handler, ff_rdt_live_audio_handler;
> lines in rtpdec.c are a good idea, but I do not know how to fix this...
> Maybe register_dynamic_payload_handler() should be exported, and we can have
> an rtd_init() function calling it to register ff_rdt_video_handler & friends?

How about $attached?

Ronald
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rtsp-dynamic_rdt_handler.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080903/b30fcafe/attachment.asc>