[FFmpeg-cvslog] r10929 - trunk/libavformat/rtp.c

[Luca Abeni]

Hi all,

lucabe wrote:
> Author: lucabe
> Date: Mon Nov  5 11:15:20 2007
> New Revision: 10929
> 
> Log:
> Use a symbolic name for the payload size of an RTCP Sender Report packet
> 
> 
> Modified:
>    trunk/libavformat/rtp.c
> 
> Modified: trunk/libavformat/rtp.c
> ==============================================================================
> --- trunk/libavformat/rtp.c	(original)
> +++ trunk/libavformat/rtp.c	Mon Nov  5 11:15:20 2007
> @@ -32,6 +32,7 @@
>  
>  //#define DEBUG
>  
> +#define RTCP_SR_SIZE 28
>  
>  /* TODO: - add RTCP statistics reporting (should be optional).
>  
> @@ -983,7 +984,7 @@ static int rtp_write_packet(AVFormatCont
>      /* XXX: mpeg pts hardcoded. RTCP send every 0.5 seconds */
>      rtcp_bytes = ((s->octet_count - s->last_octet_count) * RTCP_TX_RATIO_NUM) /
>          RTCP_TX_RATIO_DEN;
> -    if (s->first_packet || rtcp_bytes >= 28) {
> +    if (s->first_packet || rtcp_bytes >= RTCP_SR_SIZE) {
BTW, the corresponding code for the RTP demuxer (in rtp_check_and_send_back_rr())
seems to be badly broken (I really do not understand how it is supposed to work...
So I did not touch it).


				Luca