[vlc-devel] [PATCH] telx: fix -Wempty-body warning

[Thomas Guillem]

I would rather do:

#ifdef TELX_DEBUG
#   define dbg( a ) msg_Dbg a
#else
-#   define dbg( a ) 
+#   define dbg( a ) (void)
#endif

Like it is done for assert().

On Tue, Oct 29, 2019, at 15:31, Alexandre Janniaux wrote:
> ---
>  modules/codec/telx.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/modules/codec/telx.c b/modules/codec/telx.c
> index 8446e1a5f4c..6e9860308f6 100644
> --- a/modules/codec/telx.c
> +++ b/modules/codec/telx.c
> @@ -682,8 +682,10 @@ static int Decode( decoder_t *p_dec, block_t 
> *p_block )
>              /* row 0 : flags and header line */
>              b_update |= DecodePageHeaderPacket( p_dec, packet, 
> magazine );
>              if( b_update )
> +            {
>                  dbg((p_dec, "%ld --> %ld", (long int) p_block->i_pts,
>                                             (long 
> int)(p_sys->prev_pts+1500000)));
> +            }
>          }
>          else if ( row < 26 ) /* Normal Packet */
>          {
> -- 
> 2.23.0
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel