On 21/04/15 14:24, Vittorio Giovara wrote:
Improves Coverity analysis, avoding "double semicolon" CIDs. --- libavcodec/internal.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)Not sure if a `do {} while(0)` is better than just a `while(0)`. Vittorio diff --git a/libavcodec/internal.h b/libavcodec/internal.h index 6f15a78..634400f 100644 --- a/libavcodec/internal.h +++ b/libavcodec/internal.h @@ -51,13 +51,13 @@ #ifdef DEBUG # define ff_dlog(ctx, ...) av_log(ctx, AV_LOG_DEBUG, __VA_ARGS__) #else -# define ff_dlog(ctx, ...) +# define ff_dlog(ctx, ...) while(0) #endif #ifdef TRACE # define ff_tlog(ctx, ...) av_log(ctx, AV_LOG_TRACE, __VA_ARGS__) #else -# define ff_tlog(p, ...) +# define ff_tlog(ctx, ...) while(0) #endif
It is fine. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
