On Fri, 24 Jun 2016, Diego Biurrun wrote:

On Fri, Jun 24, 2016 at 01:07:08AM +0300, Martin Storsjö wrote:
--- /dev/null
+++ b/libavcodec/libopenh264.h
@@ -0,0 +1,39 @@
+
+#ifndef AVCODEC_LIBOPENH264_H
+#define AVCODEC_LIBOPENH264_H
+
+#endif

missing #endif comment

... and if I fix that?

A "LGTM otherwise" or similar (or an explicit note saying the opposite), for trivial review comments like these, would be useful, to save one extra round of resending and review...

// Martin
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to