diff options
author | Rob Landley <rob@landley.net> | 2006-09-15 08:08:25 +0400 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2006-09-15 08:08:25 +0400 |
commit | f5fc1384e97eb11bf1f0e61308554da5883209d2 (patch) | |
tree | 3900da8babd792ef2205fa2dd56b65fc2c1eb03e | |
parent | 69115c656d7b16732f11bc5e92539ef441794c03 (diff) |
So, in the cornucopia of superfulous warning directives I didn't add to the
tree, we have the warning about failed inlines (which with our inline
limit set to zero means any usage of the "inline" keyword at all). Note
that setting the inline limit to zero, and using -Werror, both predated
adding the warning about inlines to the tree. So whatever checkin added
that did nothing but break the tree. But oh well.
The second category of superfluous warnings is warning about functions with
no previous declaration. Apparently, if you add ALWAYS_INLINE to an empty
function definition, it considers the sucker undeclared as far as the
warning is concerned. (I.E. it's a buggy warning. I try not to ask the
compiler to generate warnings it can't competently generate.)
This is why I removed "inline" (unbreak allbareconfig), and couldn't replace it
with "ALWAYS_INLINE" (still broke allbareconfig).
-rw-r--r-- | networking/wget.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/networking/wget.c b/networking/wget.c index f0ef3e485..181ea9397 100644 --- a/networking/wget.c +++ b/networking/wget.c @@ -36,7 +36,7 @@ enum { STALLTIME = 5 }; #else -static inline void progressmeter(int flag) {} +static void progressmeter(int flag) {} #endif static void close_and_delete_outfile(FILE* output, char *fname_out, int do_continue) |