Also do not warn about function declarations without a prior declaration. *sigh*

[cparser] / parser.c

diff --git a/parser.c b/parser.c
index f76817c..5fa99de 100644 (file)
--- a/parser.c
+++ b/parser.c
@@ -2343,16 +2343,20 @@ warn_redundant_declaration:
                        }
                        return previous_declaration;
                }
-       } else if (is_function_definition &&
-                       declaration->storage_class != STORAGE_CLASS_STATIC) {
-               if (warning.missing_prototypes && !is_sym_main(symbol)) {
-                       warningf(declaration->source_position, "no previous prototype for '%#T'", type, symbol);
-               } else if (warning.missing_declarations && !is_sym_main(symbol)) {
-                       warningf(declaration->source_position, "no previous declaration for '%#T'", type, symbol);
+       } else if (is_function_definition) {
+               if (declaration->storage_class != STORAGE_CLASS_STATIC) {
+                       if (warning.missing_prototypes && !is_sym_main(symbol)) {
+                               warningf(declaration->source_position, "no previous prototype for '%#T'", type, symbol);
+                       } else if (warning.missing_declarations && !is_sym_main(symbol)) {
+                               warningf(declaration->source_position, "no previous declaration for '%#T'", type, symbol);
+                       }
                }
        } else if (warning.missing_declarations &&
-           declaration->storage_class != STORAGE_CLASS_STATIC &&
-           declaration->storage_class != STORAGE_CLASS_TYPEDEF) {
+           context == global_context &&
+           !is_type_function(type) && (
+             declaration->storage_class == STORAGE_CLASS_NONE ||
+             declaration->storage_class == STORAGE_CLASS_THREAD
+           )) {
                warningf(declaration->source_position, "no previous declaration for '%#T'", type, symbol);
        }