[Ffmpeg-devel] [PATCH proposal] RTCP receiver report

Ryan Martell rdm4
Mon Oct 30 18:19:51 CET 2006


Hi Thijs--

I'm having problems with this patch. When I wrote the code to send  
the receiver report, I was always getting a -2 return value from  
url_write().  I stopped looking for this problem when I saw you had  
solved this (Thanks!).

However, now that I have created the patch to insert the correct  
statistics in, I still get the -2 on url_write().

I noticed you didn't check the return value- are you sure you are  
writing correctly?

     if ((len > 0) && buf) {
         int result;
#if defined(DEBUG)
         printf("sending %d bytes of RR\n", len);
#endif
         result= url_write(s->rtp_ctx, buf, len);
#if defined(DEBUG)
         printf("result from url_write: %d\n", result);
#endif
        av_free(buf);
     }

If it's working on your end, let me know, and I'll see if I can  
figure out what's wrong on mine.

Thanks,
-Ryan

On Oct 27, 2006, at 3:06 PM, Thijs Vermeir wrote:

> Hi
> On 27-okt-06, at 20:23, Guillaume POIRIER wrote:
>
>> Hi,
>> ...
>> Patch applied, minus tabs and trailing spaces goodies.
>>
>
> Thanks,
>
>> Thijs, you need to remove all trailing spaces and tabs from your
>> files. VI has a mode for to spot them: ":set list". Emacs can remove
>> them with m-x untabify and m-x whitespace-cleanup.
>>
>
> I'm a vim user and will use this option in the next patches i post.
>
> Grts,
> Thijs
>
>> Guillaume
>> -- 
>> With DADVSI (http://en.wikipedia.org/wiki/DADVSI), France finally has
>> a lead on USA on selling out individuals right to corporations!
>> Vive la France!
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at mplayerhq.hu
>> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>





More information about the ffmpeg-devel mailing list