On 03/12/11 23:31, Nathan Caldwell wrote:
2011/12/3 Måns Rullgård<[email protected]>:
Nathan Caldwell<[email protected]>  writes:
+if (cond) { \
+    av_log(avctx, AV_LOG_ERROR, __VA_ARGS__); \
+    return AVERROR(EINVAL); \
+}

Please consider making this a do { } while (0).

Why would this be needed in this case?

Update attached.

Seems fine.

lu

--

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to