Hari Kodungallur <hari.mailinglists@xxxxxxxxx> writes: > On Mon, Dec 20, 2010 at 5:22 PM, Ian Lance Taylor <iant@xxxxxxxxxx> wrote: >> Hari Kodungallur <hari.mailinglists@xxxxxxxxx> writes: >> >>>>From <sys/stat.h>: >>> ================= >>> >>> ... >>> ... >>> ... >>> >>> #ifndef __USE_FILE_OFFSET64 >>> /* Get file attributes for FILE and put them in BUF. Â*/ >>> extern int stat (__const char *__restrict __file, >>> Â Â Â Â Â Â Â Â struct stat *__restrict __buf) __THROW __nonnull ((1, 2)); >>> >> >>>>From fixed sys/stat.h: >>> ====================== >>> >>> ... >>> ... >>> ... >>> >>> #ifndef __USE_FILE_OFFSET64 >>> /* Get file attributes for FILE and put them in BUF. Â*/ >>> #ifdef __GNUC_GNU_INLINE__ >>> extern >>> #endif >>> __inline__ int stat (__const char *__restrict __file, >>> Â Â Â Â Â Â Â Â struct stat *__restrict __buf) __THROW __nonnull ((1, 2)); >> >> This is pretty clearly a bug when using fixincludes with an older >> version of glibc. ÂAs far as I can see the bug still exists in the >> current sources. ÂCan you open a bug report for this? ÂSee >> http://gcc.gnu.org/bugs/ . Thanks. >> > > OK, I'll file a bug. Thanks much Ian. > > I'm wondering what the desired behaviour is? Keep them as "extern int"? Yes. There is no need to change the function declarations. It is only necessary to change the function definitions. Ian