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

[Diego Biurrun]

On Tue, Mar 31, 2009 at 09:47:49AM +0000, Carl Eugen Hoyos wrote:
> 
> Diego Biurrun <diego <at> biurrun.de> writes:
> 
> > On Tue, Mar 31, 2009 at 11:18:01AM +0200, gb wrote:
> > > 
> > > Log:
> > > 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.

> Additionally, you were unmistakeably asked to revert and did not so this clearly
> was a revert.

Discussions were still ongoing and Michael was simply wrong.  You know,
it happens.

> > > It seems the process is "mess the stuff around and let others fix my
> > > breakage"...
> > 
> > It clearly is not failing to understand the process and then be an
> > arrogant idiot parading your ignorance around.
> 
> While I (probably much more often than you) sometimes disagree with Gwenole, I
> would strongly ask you to treat him the same way we all repeatedly asked on these
> lists to be treated.

I'm not the one insulting others in log messages.  This is doubly
annoying because Gwenole is wrong.

Diego