[FFmpeg-cvslog] r18258 - trunk/libavcodec/vaapi.h

Carl Eugen Hoyos cehoyos
Tue Mar 31 12:27:17 CEST 2009


Diego Biurrun <diego <at> biurrun.de> writes:

> > > > Revert Diego's r18249:
> > > > Add required va/va.h header, should fix 'make checkheaders'.
> > > 
> > > This is not a revert, you can now remove that header because the file no
> > > longer depends on it.
> > 
> > Note that "make checkheaders" failed on many supported platforms with and
> > without your patch for this header, so it never was a "fix".
> 
> Fixing one header is a fix, no matter how many other headers are broken,
> just as fixing one crash is a fix, no matter how many other crasher bugs
> remain.

Just to clarify:
I believe I'm running a supported platform, and for me "make checkheaders"
failed for libavcodec/vaapi.h both with and without your patch.
And yes, I believe "fixing" a crash that afterwards still crashes on some
supported platforms is _not_ a fix.

Carl Eugen




More information about the ffmpeg-cvslog mailing list