[MPlayer-cvslog] r33790 - in trunk: configure libmpcodecs/vf_yadif.c

[Reimar Döffinger]

On 2 Jul 2011, at 14:07, Diego Biurrun <diego at biurrun.de> wrote:
> On Fri, Jul 01, 2011 at 11:57:28PM +0300, Ivan Kalvachev wrote:
>> On 7/1/11, diego <subversion at mplayerhq.hu> wrote:
>>> 
>>> Log:
>>> configure: Drop check for compiler support of named assembler arguments.
>>> 
>>> This was done to accomodate legacy compilers that are no longer supported.
>> 
>> Exactly what compilers do you have in mind and who exactly decided
>> they are not longer supported?
>> 
>> This commind, r33770 and r33786, It looks like you are intentionally
>> trying to cripple older compilers support.
> 
> False, I am doing no such thing.  I suggest you grab one of those older
> compilers (good luck getting them built in the first place) and see for
> yourself, like I did.

Huh? Before FFmpeg removed support it we through fine with compilation except for one actual  FFmpeg bug which is fixed (AFAIK not yet fixed in Libav).
Some codecs would not work properly with SSE and some MMX2 functions, but since that was never fully investigated I don't know for sure what the reason was.
And there's no need to compile, just install the recent Haiku alpha, gcc 2.95 is still the default compiler.
So you'll have to use a few more words if you want us to see anything for ourselves.
Removing that stuff when you are aware that some other developers just started playing a bit with gcc 2.95 certainly isn't the height of politeness.