From: Wolfgang (Blub) Bumiller <blub@speed.at>
Date: Sat, 18 Aug 2012 19:54:19 +0000 (+0200)
Subject: -Wlocal-shadows - note that -std=gmqcc is very different from the rest
X-Git-Tag: 0.1-rc1~174
X-Git-Url: https://git.rm.cloudns.org/?a=commitdiff_plain;h=ee3aaa58b31981065c8721a415f7482b2ff67059;p=xonotic%2Fgmqcc.git

-Wlocal-shadows - note that -std=gmqcc is very different from the rest
---

diff --git a/parser.c b/parser.c
index dbe7fa7..567a622 100644
--- a/parser.c
+++ b/parser.c
@@ -1792,6 +1792,7 @@ static bool parser_variable(parser_t *parser, ast_block *localblock)
         olddecl = NULL;
         isfunc  = false;
         func    = NULL;
+        isparam = false;
         ctx = parser_ctx(parser);
         var = parser_parse_type(parser, basetype, &isfunc);
 
@@ -1811,11 +1812,52 @@ static bool parser_variable(parser_t *parser, ast_block *localblock)
                 return false;
             }
 
-            if (localblock && (olddecl = parser_find_local(parser, parser_tokval(parser), parser->blocklocal, &isparam))) {
-                ast_value_delete(var);
-                parseerror(parser, "local `%s` already declared here: %s:%i",
-                           parser_tokval(parser), ast_ctx(olddecl).file, (int)ast_ctx(olddecl).line);
-                return false;
+            if (localblock) {
+                olddecl = parser_find_local(parser, parser_tokval(parser), parser->blocklocal, &isparam);
+                if (opts_standard == COMPILER_GMQCC)
+                {
+                    if (olddecl)
+                    {
+                        if (!isparam) {
+                            ast_value_delete(var);
+                            parseerror(parser, "local `%s` already declared here: %s:%i",
+                                       parser_tokval(parser), ast_ctx(olddecl).file, (int)ast_ctx(olddecl).line);
+                            return false;
+                        }
+                    }
+
+                    if( (!isparam && olddecl) || (olddecl = parser_find_local(parser, parser_tokval(parser), 0, &isparam)) )
+                    {
+                        if (parsewarning(parser, WARN_LOCAL_SHADOWS,
+                                         "local `%s` is shadowing a parameter", parser_tokval(parser)))
+                        {
+                            ast_value_delete(var);
+                            parseerror(parser, "local `%s` already declared here: %s:%i",
+                                       parser_tokval(parser), ast_ctx(olddecl).file, (int)ast_ctx(olddecl).line);
+                            return false;
+                        }
+                    }
+                }
+                else
+                {
+                    if (olddecl)
+                    {
+                        if (isparam &&
+                            parsewarning(parser, WARN_LOCAL_SHADOWS,
+                                         "a parameter is shadowing local `%s`", parser_tokval(parser)))
+                        {
+                            ast_value_delete(var);
+                            return false;
+                        }
+                        else if (!isparam)
+                        {
+                            ast_value_delete(var);
+                            parseerror(parser, "local `%s` already declared here: %s:%i",
+                                       parser_tokval(parser), ast_ctx(olddecl).file, (int)ast_ctx(olddecl).line);
+                            return false;
+                        }
+                    }
+                }
             }
         }
 
diff --git a/warns.def b/warns.def
index 88b4cfc..6958a99 100644
--- a/warns.def
+++ b/warns.def
@@ -9,3 +9,4 @@ GMQCC_DEFINE_FLAG(EXTENSIONS)
 GMQCC_DEFINE_FLAG(FIELD_REDECLARED)
 GMQCC_DEFINE_FLAG(MISSING_RETURN_VALUES)
 GMQCC_DEFINE_FLAG(TOO_FEW_PARAMETERS)
+GMQCC_DEFINE_FLAG(LOCAL_SHADOWS)