On Tue, 12 Mar 2013, Diego Biurrun wrote:

+int av_qsv_get_free_sync(av_qsv_space *space, av_qsv_context *qsv)
+{
+    int ret     = -1;
+    int counter = 0;
+
+    while (1) {
+        for (int i = 0; i < space->sync_num; i++)

Don't declare variables inside for statements like this.  Some compilers
choke on it.  Supposedly also MSVC, so I wonder why you did not stumble
over the problem.  You have a new MSVC version that does not trip up?

The c99-to-c89 converter handles these nowadays. But nevertheless, we should try to keep our codebase free from them since we don't have any of them so far.

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

Reply via email to