DEADSOFTWARE

Исправен конфликт имён локальных и глобальных процедур
authorDeaDDooMER <deaddoomer@deadsoftware.ru>
Wed, 27 Sep 2017 17:28:08 +0000 (20:28 +0300)
committerDeaDDooMER <deaddoomer@deadsoftware.ru>
Wed, 27 Sep 2017 17:28:08 +0000 (20:28 +0300)
Test.obn
obn-run-tests.sh
src/oberon.c
tests/Test29.obn [new file with mode: 0644]

index 6fae5299c4ceb508ddf3708622d39e419e562a12..da60894a8f758fa1ea09b7ef7f8195caa7367b55 100644 (file)
--- a/Test.obn
+++ b/Test.obn
@@ -1,3 +1,11 @@
 MODULE Test;
 
+PROCEDURE A;
+END A;
+
+PROCEDURE B;
+  PROCEDURE A;
+  END A;
+END B;
+
 END Test.
index 8db6f37b203c5a6c9733df4dc90f79b6dc0f8e2e..2ce75728bd40a4f5c5fb3d3ef6a41f256444f15d 100755 (executable)
@@ -101,3 +101,4 @@ maketest Test25
 maketest Test26
 maketest Test27
 maketest Test28
+maketest Test29
index cd7e97a18f3cb5531e77bd28d1253a62cdc92b85..82566471b9a11b929d832f2d51ef355aa4d23dfc 100644 (file)
@@ -305,6 +305,21 @@ oberon_find_object(oberon_scope_t * scope, char * name, bool check_it)
        return result;
 }
 
+static oberon_object_t *
+oberon_find_object_in_scope(oberon_scope_t * scope, char * name, bool check_it)
+{
+       oberon_object_t * result = NULL;
+
+       result = oberon_find_object_in_list(scope -> list, name);
+
+       if(check_it && result == NULL)
+       {
+               oberon_error(scope -> ctx, "undefined ident %s", name);
+       }
+
+       return result;
+}
+
 static oberon_object_t *
 oberon_create_object(oberon_scope_t * scope, char * name, int class, bool export, bool read_only)
 {
@@ -2469,7 +2484,7 @@ oberon_proc_decl(oberon_context_t * ctx)
        oberon_close_scope(ctx -> decl);
 
        oberon_object_t * proc;
-       proc = oberon_find_object(ctx -> decl, name, 0);
+       proc = oberon_find_object_in_scope(ctx -> decl, name, 0);
        if(proc == NULL)
        {
                proc = oberon_define_object(ctx -> decl, name, OBERON_CLASS_PROC, export, read_only, false);
diff --git a/tests/Test29.obn b/tests/Test29.obn
new file mode 100644 (file)
index 0000000..ab7e493
--- /dev/null
@@ -0,0 +1,13 @@
+MODULE Test29;
+
+PROCEDURE A;
+END A;
+
+PROCEDURE B;
+  PROCEDURE A;
+  END A;
+END B;
+
+END Test29.
+
+Объявление локальной процедуры не должно конфликтовать с глобальным.