[FFmpeg-devel] [PATCH] zerocodec: fix direct rendering.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Jul 22 22:54:58 CEST 2012


On Sun, Jul 22, 2012 at 04:25:25PM -0400, Derek Buitenhuis wrote:
> On 22/07/2012 4:18 PM, Reimar Döffinger wrote:
> > I am generally in favour of decoding as well as possible, though to
> > be honest that would include not failing for a missing reference but
> > instead using a black reference.
> > At the same time, we now have decode_error_flags we could use to signal
> > a (potential) error until the next keyframe while still decoding the
> > frames.
> > We might need to discuss how exactly to use it though...
> 
> I'm not sure it's worth the effort. Nobody actually uses ZeroCodec.
> 
> As it stands, our code is already more robust than the upstream decoder,
> which just crashes all the time and doesn't check colorspace, etc.
> 
> Just my two cents.

Sure, just leave it if you want.
I just kind of like doing these kind of things due to
1) It allows testing API things in a simple case that can be properly
understood
2) Those simple codecs are often used as a base for new, more complex
ones where it might be worth it

So if someone thinks it would be fun to do this 100% I'll happily
discuss it more, but I am not _asking_ anyone to do it.


More information about the ffmpeg-devel mailing list