[04/11] rtpdec_h264: Remove a useless ifdef
diff mbox

Message ID 1336169215-59789-4-git-send-email-martin@martin.st
State Committed
Commit 44f99fe0f5e688833cb671efbaa0beb6b74c9389
Headers show

Commit Message

Martin Storsjö May 4, 2012, 10:06 p.m. 
assert is a no-op if DEBUG isn't defined.
---
 libavformat/rtpdec_h264.c |    2 --
 1 file changed, 2 deletions(-)

Comments

Luca Barbato May 4, 2012, 11:58 p.m. | #1 
On 04/05/12 15:06, Martin Storsjö wrote:
> assert is a no-op if DEBUG isn't defined.
> ---
>  libavformat/rtpdec_h264.c |    2 --
>  1 file changed, 2 deletions(-)
> 

Ok.
Martin Storsjö May 5, 2012, 12:16 a.m. | #2 
On Fri, 4 May 2012, Luca Barbato wrote:

> On 04/05/12 15:06, Martin Storsjö wrote:
>> assert is a no-op if DEBUG isn't defined.
>> ---
>>  libavformat/rtpdec_h264.c |    2 --
>>  1 file changed, 2 deletions(-)
>> 
>
> Ok.

Pushed the cleanup patches up to this one, I'll do another version of the 
comment cleanup and/or awaiting more opinions on it.

// Martin

Patch
diff mbox 

diff --git a/libavformat/rtpdec_h264.c b/libavformat/rtpdec_h264.c
index d98be07..399ca47 100644
--- a/libavformat/rtpdec_h264.c
+++ b/libavformat/rtpdec_h264.c
@@ -170,9 +170,7 @@  static int h264_handle_packet(AVFormatContext *ctx,
     nal  = buf[0];
     type = nal & 0x1f;
 
-#ifdef DEBUG
     assert(data);
-#endif
     assert(buf);
 
     if (type >= 1 && type <= 23)