Oops. Bad scoping for iterators and switch() statements. We didn't

close the scope properly at the end (we started a new one instead).

Make all non-static functions external.

And always check the declarator against previous symbols when finding
a new symbol. We want to verify that we don't have duplicates.

1 files changed, 18 insertions, 0 deletions

diff --git a/symbol.c b/symbol.c
index 670fd976..2c56ad15 100644
--- a/symbol.c
+++ b/symbol.c
@@ -267,6 +267,24 @@ struct symbol *examine_symbol_type(struct symbol * sym)
 	return sym;
 }
 
+void check_declaration(struct symbol *sym)
+{
+	struct symbol *next = sym;
+
+	while ((next = next->next_id) != NULL) {
+		if (next->namespace != sym->namespace)
+			continue;
+		if (sym->scope == next->scope) {
+			sym->same_symbol = next;
+			return;
+		}
+		if (sym->ctype.modifiers & next->ctype.modifiers & MOD_EXTERN) {
+			sym->same_symbol = next;
+			return;
+		}
+	}
+}
+
 void bind_symbol(struct symbol *sym, struct ident *ident, enum namespace ns)
 {
 	struct scope *scope;