Close ASDL schemas so that descriptors aren't leaking.

They were taking up descriptors 5 and 6 which might have been a problem.
oilshell · Jul 2, 2017 · 191a487505f44eaca14a7531931c90f8d53fb2c1 · 191a487
1 parent e8dbf85
commit 191a487505f44eaca14a7531931c90f8d53fb2c1
Unified Split

Showing with 6 additions and 2 deletions.

+1 −1 core/builtin.py

+1 −0 core/runtime.py

+1 −0 osh/ast_.py

+3 −1 test/wild2.sh
diff --git a/core/builtin.py b/core/builtin.py
@@ -699,7 +699,7 @@ def Set(argv, exec_opts, mem):
  if not argv:  # empty
    # TODO: If no arguments are given, it shows functions/vars?  Why not show
    # other state?
-    print('TODO')
+    print('TODO: set without arguments')
    return 0

  arg, i = set_spec.Parse(argv)
diff --git a/core/runtime.py b/core/runtime.py
@@ -27,3 +27,4 @@ def _ParseAndMakeTypes(f, root):
 f = util.GetResourceLoader().open('core/runtime.asdl')
 root = sys.modules[__name__]
 _ParseAndMakeTypes(f, root)
+f.close()
diff --git a/osh/ast_.py b/osh/ast_.py
@@ -140,3 +140,4 @@ def _ParseAndMakeTypes(f, root):
 f = util.GetResourceLoader().open('osh/osh.asdl')
 root = sys.modules[__name__]
 _ParseAndMakeTypes(f, root)
+f.close()
diff --git a/test/wild2.sh b/test/wild2.sh
@@ -91,7 +91,9 @@ sh-config-status() {
  local out=_tmp/wild2/$(basename $sh)-py-configure

  pushd $out
-  $sh -x ./config.status
+  #$sh -x ./config.status
+  $sh ./config.status
+  #strace -e dup2,fcntl -- $sh ./config.status
  popd
  echo status=$?