Re: CVS commit: src/sys/fs/udf

[Jason Thorpe <thorpej%shagadelic.org@localhost>]

On Aug 10, 2006, at 5:26 AM, Reinoud Zandijk wrote:

> Module Name:    src
> Committed By:   reinoud
> Date:           Thu Aug 10 12:26:44 UTC 2006
>
> Modified Files:
>         src/sys/fs/udf: udf_osta.c
>
> Log Message:
> The local variable `index' seems shadowing an inbuild gcc variable  
> (???)
> when compiling from userland (kernel ok) so rename the variable to  
> Index to
> avoid the confusing warning.

It's not all that confusing -- "index" is the name of a C library  
function that GCC presumably has a built-in implementation for (for  
literal strings constants, most likely).  The kernel doesn't generate  
this warning because it is built with options that tell the compiler  
"not in a hosted environment", which disables automatic expansion of  
certain functions into built-ins.

BTW, why not rename the variable to "idx" so you can avoid the StUdLy  
CaPs?

-- thorpej