Merge pull request #41 from zopefoundation/revert-11-master

Revert "Add built-in function 'set'"
zopefoundation · Mar 2, 2017 · da3c2f1 · da3c2f1
2 parents b62759a + c24724f
commit da3c2f1
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 10 deletions.
diff --git a/src/RestrictedPython/Guards.py b/src/RestrictedPython/Guards.py
@@ -23,7 +23,7 @@
 for name in ['False', 'None', 'True', 'abs', 'basestring', 'bool', 'callable',
              'chr', 'cmp', 'complex', 'divmod', 'float', 'hash',
              'hex', 'id', 'int', 'isinstance', 'issubclass', 'len',
-             'long', 'oct', 'ord', 'pow', 'range', 'repr', 'round', 'set',
+             'long', 'oct', 'ord', 'pow', 'range', 'repr', 'round',
              'str', 'tuple', 'unichr', 'unicode', 'xrange', 'zip']:
 
     safe_builtins[name] = __builtins__[name]

diff --git a/src/RestrictedPython/tests/restricted_module.py b/src/RestrictedPython/tests/restricted_module.py
@@ -33,11 +33,6 @@ def try_map():
     print map(inc, x),
     return printed
 
-def try_set():
-    x = [1, 1, 2, 3]
-    print set(x)
-    return printed
-
 def try_apply():
     def f(x, y, z):
         return x + y + z

diff --git a/src/RestrictedPython/tests/testRestrictions.py b/src/RestrictedPython/tests/testRestrictions.py
@@ -251,10 +251,6 @@ def checkTryMap(self):
         res = self.execFunc('try_map')
         self.assertEqual(res, "[2, 3, 4]")
 
-    def checkTrySet(self):
-        res = self.execFunc('try_set')
-        self.assertEqual(res, "set([1, 2, 3])")
-
     def checkApply(self):
         del apply_wrapper_called[:]
         res = self.execFunc('try_apply')