On Sun, Feb 1, 2009 at 11:30 PM, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote:
> Anyway, proposed patch for (1) follows:
I read the patch, seems reasonable. It is only solve the inline case though.
The more generic problem still exist, symbol look up between partial
prototype declare and the real declare will get symbol with partial
information.
It would be great if we can apply the definition to the first symbol. But
I can haven't find a clean way to do it yet.
Most likely I will apply your patch until we find a better way to do it.
> @@ -2282,6 +2295,7 @@ struct token *external_declaration(struct token *token, struct symbol_list **lis
> }
> }
> } else if (base_type && base_type->type == SYM_FN) {
> + if (decl->next_id && decl->next_id->scope == decl->scope)
Not sure what this is trying to do. Shouldn't the next line to be indented
if I read it correctly?
Chris
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
