rtsp: Set the default delay to 0.1 s for the RTSP/SDP/RTP demuxers
authorMartin Storsjö <martin@martin.st>
Mon, 19 Mar 2012 18:40:23 +0000 (20:40 +0200)
committerMartin Storsjö <martin@martin.st>
Tue, 20 Mar 2012 08:53:49 +0000 (10:53 +0200)
This enables reordering of UDP packets by default, unless the caller
explicitly sets -max_delay 0.

Signed-off-by: Martin Storsjö <martin@martin.st>
doc/protocols.texi
libavformat/rtsp.c

index 0eb4c69..b08233e 100644 (file)
@@ -288,9 +288,9 @@ Accept packets only from negotiated peer address and port.
 @end table
 
 When receiving data over UDP, the demuxer tries to reorder received packets
-(since they may arrive out of order, or packets may get lost totally). In
-order for this to be enabled, a maximum delay must be specified in the
-@code{max_delay} field of AVFormatContext.
+(since they may arrive out of order, or packets may get lost totally). This
+can be disabled by setting the maximum demuxing delay to zero (via
+the @code{max_delay} field of AVFormatContext).
 
 When watching multi-bitrate Real-RTSP streams with @file{avplay}, the
 streams to display can be chosen with @code{-vst} @var{n} and
index 7afd106..07cf809 100644 (file)
@@ -56,6 +56,7 @@
 #define MAX_TIMEOUTS READ_PACKET_TIMEOUT_S * 1000 / POLL_TIMEOUT_MS
 #define SDP_MAX_SIZE 16384
 #define RECVBUF_SIZE 10 * RTP_MAX_PACKET_LENGTH
+#define DEFAULT_REORDERING_DELAY 100000
 
 #define OFFSET(x) offsetof(RTSPState, x)
 #define DEC AV_OPT_FLAG_DECODING_PARAM
@@ -1427,7 +1428,7 @@ int ff_rtsp_connect(AVFormatContext *s)
         return AVERROR(EIO);
 
     if (s->max_delay < 0) /* Not set by the caller */
-        s->max_delay = 0;
+        s->max_delay = s->iformat ? DEFAULT_REORDERING_DELAY : 0;
 
     rt->control_transport = RTSP_MODE_PLAIN;
     if (rt->lower_transport_mask & (1 << RTSP_LOWER_TRANSPORT_HTTP)) {
@@ -1870,7 +1871,7 @@ static int sdp_read_header(AVFormatContext *s)
         return AVERROR(EIO);
 
     if (s->max_delay < 0) /* Not set by the caller */
-        s->max_delay = 0;
+        s->max_delay = DEFAULT_REORDERING_DELAY;
 
     /* read the whole sdp file */
     /* XXX: better loading */