On Thu, Aug 10, 2017 at 09:13:13AM +0200, Nicolas George wrote:
> Le tridi 23 thermidor, an CCXXV, Clement Boesch a écrit :
> > > -        .needs_writable = 1, // for error diffusal dithering
> > why?
> It was not needed in the first place: IIRC, dualinput always returns a
> writable main frame. And the new implementation does, I am sure of it.

But unless it's API documented, that's implementation specific. I'd prefer
if you keep that as a safe guard. It also has a documentation purpose. If
the frame is already writable it will be a noop.

Clément B.

Attachment: signature.asc
Description: PGP signature

ffmpeg-devel mailing list

Reply via email to