Niklas Haas (HE12025-08-05):
> I will repeat myself briefly here that such a flag should be an opt-out, not
> an opt-in. Supporting premultiplied alpha should be the default assumption
> unless provable otherwise.

Absolutely not.

Do you dereference a pointer when you can prove it cannot be NULL or
when you cannot prove it can be NULL? It is the same: if you cannot
prove a filter produces a valid result, then we protect by default. It
is already too much that it is not blocking when it is unspecified. On
this I will stand firm.

> On a side note, I don't think it's necessary to add a new "default" struct
> just for one particular combination of flags;

Of course it is not necessary, it just makes a dozen other patches much
much shorter.

>                                               especially not if your intention
> is (as you state) to have such flags for every other AVFrame property.

It does not make it either harder nor easier.

-- 
  Nicolas George
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to