Merge pull request #854 from kachick/fix-some-defined

Guard `defined?` from segfault with default value

tests/test_interpreter.py

@@ -1105,6 +1105,10 @@ def test_defined(self, space):
         """)
         assert space.str_w(w_res) == "expression"
         w_res = space.execute("""
+        return [defined?(while x do y end), defined?(return), defined?(__FILE__)]
+        """)
+        assert self.unwrap(space, w_res) == ["expression", "expression", "expression"]
+        w_res = space.execute("""
         $abc = 3
         return [defined?($abc), defined?($abd)]
         """)

topaz/ast.py

@@ -21,6 +21,9 @@ def compile(self, ctx):
         else:
             raise NotImplementedError(type(self).__name__)
 
+    def compile_defined(self, ctx):
+        ConstantString("expression").compile(ctx)
+
 
 class Node(BaseNode):
     _attrs_ = ["lineno"]