On 09/13/2013 04:46 PM, Luca Barbato wrote:
On 13/09/13 22:30, Justin Ruggles wrote:
+            s->alpha_gb = gb;
+            bytestream2_skip(&gb, chunk_size);
+            alpha_header = bytestream2_get_byte(&s->alpha_gb);
+
+            pre_p       = (alpha_header >> 4) & 0x03;
+            filter_m    = (alpha_header >> 2) & 0x03;
+            compression =  alpha_header       & 0x03;
+
+            if (filter_m || compression) {
+                av_log(avctx, AV_LOG_VERBOSE,
+                       "skipping unsupported ALPHA chunk\n");
+            } else {
+                s->has_alpha = 1;
+            }

Drop braces, what if you have has_alpha set but filter or compression on?

I believe it will just not decode the alpha channel, but I'll double-check.

-Justin

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

Reply via email to