[FFmpeg-devel] [PATCH+RFC] Sort of Cosmetic: index is a gcc builtin now
Sat Jul 10 20:41:12 CEST 2010
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?
mans at mansr.com
More information about the ffmpeg-devel