[FFmpeg-devel] [PATCH+RFC] Sort of Cosmetic: index is a gcc builtin now

[Måns Rullgård]

Luca Barbato <lu_zero at gentoo.org> writes:

> On 07/09/2010 11:58 PM, M?ns Rullg?rd wrote:
>> Luca Barbato <lu_zero at gentoo.org> writes:
>> 
>>> index(3) is a libc function that now is part of the gcc builtins, that
>>> makes a bit annoying debugging since gdb now tells you it is a builtin
>>> instead of printing the value you'd expect.
>> 
>> Sounds like a gdb bug.  Is the generated code still correct?
>
> The generated code looks correct to me, still I'm not sure if that might
> change sooner or later. Still we include strings.h already in some
> places so we should pay attention even if the compiler will error out in
> most common cases.

Symbols with local scope always override those with global scope, and
the name 'index' not reserved at all.  If gcc ever starts treating it
differently, that is a huge bug.

I'm mildly against renaming things only due to gdb bugs.  Have you
filed a bug report against gdb?

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