[FFmpeg-cvslog] r17615 - in trunk/libavformat: rdt.c rtpdec.h

rbultje subversion
Thu Feb 26 15:23:05 CET 2009


Author: rbultje
Date: Thu Feb 26 15:23:05 2009
New Revision: 17615

Log:
Don't (ab)use PKT_FLAG_* in RTP code, since the two have virtually nothing
in common except for this one value. Change was requested by Luca in the
"[FFmpeg-devel] RTP mark bit not passed to parse_packet" thread.

Modified:
   trunk/libavformat/rdt.c
   trunk/libavformat/rtpdec.h

Modified: trunk/libavformat/rdt.c
==============================================================================
--- trunk/libavformat/rdt.c	Thu Feb 26 15:15:41 2009	(r17614)
+++ trunk/libavformat/rdt.c	Thu Feb 26 15:23:05 2009	(r17615)
@@ -306,7 +306,7 @@ rdt_parse_packet (AVFormatContext *ctx, 
         int pos;
 
         init_put_byte(&pb, buf, len, 0, NULL, NULL, NULL, NULL);
-        flags = (flags & PKT_FLAG_KEY) ? 2 : 0;
+        flags = (flags & RTP_FLAG_KEY) ? 2 : 0;
         res = ff_rm_parse_packet (rdt->rmctx, &pb, st, rdt->rmst[st->index], len, pkt,
                                   &seq, &flags, timestamp);
         pos = url_ftell(&pb);
@@ -361,7 +361,7 @@ ff_rdt_parse_packet(RDTDemuxContext *s, 
     if (is_keyframe &&
         (set_id != s->prev_set_id || timestamp != s->prev_timestamp ||
          stream_id != s->prev_stream_id)) {
-        flags |= PKT_FLAG_KEY;
+        flags |= RTP_FLAG_KEY;
         s->prev_set_id    = set_id;
         s->prev_timestamp = timestamp;
     }

Modified: trunk/libavformat/rtpdec.h
==============================================================================
--- trunk/libavformat/rtpdec.h	Thu Feb 26 15:15:41 2009	(r17614)
+++ trunk/libavformat/rtpdec.h	Thu Feb 26 15:23:05 2009	(r17615)
@@ -92,6 +92,7 @@ typedef struct {
     uint32_t jitter;            ///< estimated jitter.
 } RTPStatistics;
 
+#define RTP_FLAG_KEY    0x1 ///< RTP packet contains a keyframe
 /**
  * Packet parsing for "private" payloads in the RTP specs.
  *
@@ -102,7 +103,7 @@ typedef struct {
  * @param timestamp pointer in which to write the timestamp of this RTP packet
  * @param buf pointer to raw RTP packet data
  * @param len length of buf
- * @param flags flags from the RTP packet header (PKT_FLAG_*)
+ * @param flags flags from the RTP packet header (RTP_FLAG_*)
  */
 typedef int (*DynamicPayloadPacketHandlerProc) (AVFormatContext *ctx,
                                                 PayloadContext *s,




More information about the ffmpeg-cvslog mailing list