Changeset 6774

Timestamp:

Apr 23, 2010, 11:46:11 AM (15 years ago)

Author:

rgrieder

Message:

Activated Strict.lua
That means you can no longer assign to or access an undeclared global variable on function scope.
Declaring: either assign anything to the variable (including nil) on file scope
or use the function global("myVar", "myVar2", …, "myVarN") (mind the strings!)

Location:

code/branches/gamestates3

Files:

• data/lua/Strict.lua (modified) (3 diffs)
• src/libraries/core/LuaState.cc (modified) (2 diffs)
• src/libraries/core/LuaState.h (modified) (1 diff)
• src/orxonox/items/MultiStateEngine.cc (modified) (1 diff)

code/branches/gamestates3/data/lua/Strict.lua

@@ -13 +13 @@
 end
 
-__STRICT = false
+__STRICT = true
 mt.__declared = {}
 
 mt.__newindex = function (t, n, v)
-  if __STRICT and not mt.__declared[n] then
-    local d = debug.getinfo(2, "S")
-    local w = d and d.what or "C"
-    if w ~= "main" and w ~= "C" then
-      error("assign to undeclared variable '"..n.."'", 2)
+  if not mt.__declared[n] then
+    if __STRICT then
+      local d = debug.getinfo(2, "S")
+      local w = d and d.what or "C"
+      if w ~= "main" and w ~= "C" then
+        error("Assigning to undeclared global variable '"..n.."'", 2)
+      end
     end
     mt.__declared[n] = true
@@ -29 +31 @@
   
 mt.__index = function (t, n)
-  if not mt.__declared[n] and debug.getinfo(2, "S").what ~= "C" then
-    error("variable '"..n.."' is not declared", 2)
+  if not mt.__declared[n] then
+    local d = debug.getinfo(2, "S")
+    local w = d and d.what or "C"
+    if w ~= "C" then
+      error("Global variable '"..n.."' was not declared", 2)
+    else
+      mt.__declared[n] = true
+    end
   end
   return rawget(t, n)
@@ -36 +44 @@
 
 function global(...)
-   for _, v in ipairs{...} do mt.__declared[v] = true end
+  for _, v in ipairs{...} do
+    mt.__declared[v] = true
+  end
 end

code/branches/gamestates3/src/libraries/core/LuaState.cc

@@ -53 +53 @@
     DeclareToluaInterface(Core);
 
-    LuaState::LuaState()
+    LuaState::LuaState(bool bStrict)
         : bIsRunning_(false)
         , includeParseFunction_(NULL)
@@ -82 +82 @@
         tolua_pushusertype(luaState_, static_cast<void*>(this), "orxonox::LuaState");
         lua_setglobal(luaState_, "luaState");
+
+        // Strict.lua ensures that global variables are not declared inside a function scope
+        if (bStrict)
+        {
+            if (!this->doFile("Strict.lua"))
+                ThrowException(InitialisationFailed, "Running Strict.lua failed");
+        }
+        else
+        {
+            // Add dummy function for declaring global variables
+            this->doString("global = function(...) end");
+        }
 
         // Parse init script

code/branches/gamestates3/src/libraries/core/LuaState.h

@@ -68 +68 @@
     { // tolua_export
     public:
-        LuaState();
+        LuaState(bool bStrict = true);
         ~LuaState();
 

code/branches/gamestates3/src/orxonox/items/MultiStateEngine.cc

@@ -61 +61 @@
             this->defEngineSndNormal_->setLooping(true);
             this->defEngineSndBoost_->setLooping(true);
-            this->lua_ = new LuaState();
+            this->lua_ = new LuaState(false);
         }
         else