Am 29.11.2017 um 13:22 schrieb Moritz Barsnick:
On Wed, Nov 29, 2017 at 00:34:03 +0000, Mark Thompson wrote:
+    if (type == AV_HWDEVICE_TYPE_NONE) {
+        fprintf(stderr, "Device type %s is not supported.\n", argv[1]);
+        fprintf(stderr, "Available device types:");
+        type = AV_HWDEVICE_TYPE_NONE;
+        while((type = av_hwdevice_iterate_types(type)) != 
+            fprintf(stderr, " %s", av_hwdevice_get_type_name(type));

I see what you're trying to do with "type = AV_HWDEVICE_TYPE_NONE"
(initialize the iterator), but the assignment is redundant.

I guess if the compiler optimizes it away anyway, it may be good for
readability, especially since it's an example. *shrug*

I fail to see how the assignment is redundant?
Without the assignment, that would be an infinite loop.

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

ffmpeg-devel mailing list

Reply via email to