[FFmpeg-devel] Realmedia patch

Ronald S. Bultje rsbultje
Mon Sep 1 00:47:51 CEST 2008


Hi,

On Sun, Aug 31, 2008 at 3:40 PM, Luca Abeni <lucabe72 at email.it> wrote:
> Ronald S. Bultje wrote:
> [...]
>> Index: ffmpeg-svn/libavformat/rdt.c
>> ===================================================================
>> --- ffmpeg-svn.orig/libavformat/rdt.c 2008-08-30 17:31:52.000000000 -0400
>> +++ ffmpeg-svn/libavformat/rdt.c      2008-08-30 18:03:32.000000000 -0400
>> @@ -28,11 +28,18 @@
>>  #include "avformat.h"
> [...]
>> +
>> +/*
>> + * RTP integration.
>> + */
>
> Sorry, I notice it only now, but... What does this comment mean?
> As it is now, it looks useless and confusing. So, either remove it
> or clarify it a little bit.

Hm... I removed it locally, the patches I just sent are still off but
it's gone locally.

It was supposed to mean "this code handles integration with
RTPDynamicPayloadHandler machinery", but that's obvious enough by
itself so it can be removed.

Ronald




More information about the ffmpeg-devel mailing list