[FFmpeg-cvslog] rtpdec_h264: Cleanup debug packet type counting

Martin Storsjö git at videolan.org
Sun May 6 22:18:57 CEST 2012


ffmpeg | branch: master | Martin Storsjö <martin at martin.st> | Sat May  5 00:28:25 2012 +0300| [48666c2bd6791e989cdb61f8fecd8dd2ce13bc58] | committer: Martin Storsjö

rtpdec_h264: Cleanup debug packet type counting

Signed-off-by: Martin Storsjö <martin at martin.st>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=48666c2bd6791e989cdb61f8fecd8dd2ce13bc58
---

 libavformat/rtpdec_h264.c |   18 +++++++++---------
 1 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/libavformat/rtpdec_h264.c b/libavformat/rtpdec_h264.c
index 27ab7be..5fd4efc 100644
--- a/libavformat/rtpdec_h264.c
+++ b/libavformat/rtpdec_h264.c
@@ -57,6 +57,12 @@ struct PayloadContext {
 #endif
 };
 
+#ifdef DEBUG
+#define COUNT_NAL_TYPE(data, nal) data->packet_types_received[(nal) & 0x1f]++
+#else
+#define COUNT_NAL_TYPE(data, nal) do { } while (0)
+#endif
+
 static int sdp_parse_fmtp_config_h264(AVStream *stream,
                                       PayloadContext *h264_data,
                                       char *attr, char *value)
@@ -188,9 +194,7 @@ static int h264_handle_packet(AVFormatContext *ctx, PayloadContext *data,
         av_new_packet(pkt, len + sizeof(start_sequence));
         memcpy(pkt->data, start_sequence, sizeof(start_sequence));
         memcpy(pkt->data + sizeof(start_sequence), buf, len);
-#ifdef DEBUG
-        data->packet_types_received[nal & 0x1f]++;
-#endif
+        COUNT_NAL_TYPE(data, nal);
         break;
 
     case 24:                   // STAP-A (one packet, multiple nals)
@@ -224,9 +228,7 @@ static int h264_handle_packet(AVFormatContext *ctx, PayloadContext *data,
                             memcpy(dst, start_sequence, sizeof(start_sequence));
                             dst += sizeof(start_sequence);
                             memcpy(dst, src, nal_size);
-#ifdef DEBUG
-                            data->packet_types_received[*src & 0x1f]++;
-#endif
+                            COUNT_NAL_TYPE(data, *src);
                             dst += nal_size;
                         }
                     } else {
@@ -287,10 +289,8 @@ static int h264_handle_packet(AVFormatContext *ctx, PayloadContext *data,
             buf++;
             len--;
 
-#ifdef DEBUG
             if (start_bit)
-                data->packet_types_received[nal_type]++;
-#endif
+                COUNT_NAL_TYPE(data, nal_type);
             if (start_bit) {
                 /* copy in the start sequence, and the reconstructed nal */
                 av_new_packet(pkt, sizeof(start_sequence) + sizeof(nal) + len);



More information about the ffmpeg-cvslog mailing list