<div dir='auto'><div>LGTM<br><div class="gmail_extra"><br><div class="gmail_quote">Le 9 nov. 2019 4:27 PM, Johan Gunnarsson <johan.gunnarsson@gmail.com> a écrit :<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Make sure the dbg macro can't compile down to nothing, so it can be used in<br>
if statements without braces.</p>
<p dir="ltr">codec/telx.c: In function ‘Decode’:<br>
codec/telx.c:686:81: warning: suggest braces around empty body in an ‘if’<br>
statement [-Wempty-body]<br>
(long int)(p_sys->prev_pts+1500000)));<br>
^<br>
---<br>
modules/codec/telx.c | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)</p>
<p dir="ltr">diff --git a/modules/codec/telx.c b/modules/codec/telx.c<br>
index 8446e1a5f4..dc5259f4e2 100644<br>
--- a/modules/codec/telx.c<br>
+++ b/modules/codec/telx.c<br>
@@ -40,7 +40,7 @@<br>
#ifdef TELX_DEBUG<br>
# define dbg( a ) msg_Dbg a<br>
#else<br>
-# define dbg( a )<br>
+# define dbg( a ) (void) 0<br>
#endif<br>
<br>
/*****************************************************************************<br>
-- <br>
2.17.1</p>
<p dir="ltr">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
https://mailman.videolan.org/listinfo/vlc-devel</p>
</blockquote></div><br></div></div></div>