[FFmpeg-devel] [PATCH] Remove useless preprocessor directives.

[Måns Rullgård]

Michael Niedermayer <michaelni at gmx.at> writes:

> On Fri, Jun 18, 2010 at 08:51:29AM +0200, Benoit Fouet wrote:
>> On Thu, 17 Jun 2010 21:57:09 +0200 Reimar D?ffinger wrote:
>> > On Thu, Jun 17, 2010 at 03:32:02PM +0200, Benoit Fouet wrote:
> [...]
>> > If you don't need/want the whole function, then put the whole function under
>> > the #if.
>> 
>> that wouldn't work either... as the code using it is in the form of:
>> if (HAVE_7REGS)
>>     foo = yuva420_rgb32_MMX;
>> You'll get an undeclared function error.
>
> what if you add a weak attibute at the function declaration?
> (should in theory work though probably its not portable enough
>  if it does work)

Weak symbols are only available on some systems.  Since we aim to work
everywhere, relying on them is not a good idea.

-- 
M?ns Rullg?rd
mans at mansr.com