aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAl Viro <viro@ftp.linux.org.uk>2009-03-09 23:32:16 +0000
committerChristopher Li <sparse@chrisli.org>2009-07-18 05:30:09 +0000
commitc14911db43a7229ba7a28ac23f4d1e569a2586ab (patch)
treecdc075ecc76deb111f4d767c74aea8e1b3c5b782
parent1aae3184cb1dbf95e1e07b4ac85cd3c70784fd32 (diff)
downloadsparse-c14911db43a7229ba7a28ac23f4d1e569a2586ab.tar.gz
Fix enumeration constants' scope beginning
It starts after the end of enumerator; i.e. if we have enum { ... Foo = expression, ... }; the scope of Foo starts only after the end of expression. Rationale: 6.2.1p7. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: Christopher Li <sparse@chrisli.org>
-rw-r--r--parse.c7
-rw-r--r--validation/enum_scope.c11
2 files changed, 14 insertions, 4 deletions
diff --git a/parse.c b/parse.c
index 4b75637f..9fb606ab 100644
--- a/parse.c
+++ b/parse.c
@@ -804,10 +804,6 @@ static struct token *parse_enum_declaration(struct token *token, struct symbol *
struct token *next = token->next;
struct symbol *sym;
- sym = alloc_symbol(token->pos, SYM_NODE);
- bind_symbol(sym, token->ident, NS_SYMBOL);
- sym->ctype.modifiers &= ~MOD_ADDRESSABLE;
-
if (match_op(next, '=')) {
next = constant_expression(next->next, &expr);
lastval = get_expression_value(expr);
@@ -828,6 +824,9 @@ static struct token *parse_enum_declaration(struct token *token, struct symbol *
expr->ctype = ctype;
}
+ sym = alloc_symbol(token->pos, SYM_NODE);
+ bind_symbol(sym, token->ident, NS_SYMBOL);
+ sym->ctype.modifiers &= ~MOD_ADDRESSABLE;
sym->initializer = expr;
sym->enum_member = 1;
sym->ctype.base_type = parent;
diff --git a/validation/enum_scope.c b/validation/enum_scope.c
new file mode 100644
index 00000000..92ffc8ef
--- /dev/null
+++ b/validation/enum_scope.c
@@ -0,0 +1,11 @@
+enum {A = 12};
+
+static void f(void)
+{
+ enum {A = A + 1, B};
+ char s[1 - 2 * (B != 14)];
+}
+
+/*
+ * check-name: enumeration constants' scope [6.2.1p7]
+ */