Try to close more resouce on windows

Carreau · Carreau · commit c1a25ff3d0bf · 2025-02-18T13:42:48.000+01:00
diff --git a/IPython/utils/_process_common.py b/IPython/utils/_process_common.py
@@ -97,6 +97,22 @@ def process_handler(
         sys.stderr.flush()
         out = None
     finally:
+        # p.stdin / p.stderr / p.stdin are not the values passes to Pope, They
+        # are proxy reader/writer, and this is usually  close in the context
+        # manager, but the __exit__ does self.wait(), and we can't
+        # afford this (see comment below),
+        # so we copy the first part of the exit routine.
+        #
+        if p.stdout:
+            p.stdout.close()
+        if p.stderr:
+            p.stderr.close()
+        try:  # Flushing a BufferedWriter may raise an error
+            if p.stdin:
+                p.stdin.close()
+        except (Exception, KeyboardInterrupt):
+            print("Error closing Stdin")
+
         # Make really sure that we don't leave processes behind, in case the
         # call above raises an exception
         # We start by assuming the subprocess finished (to avoid NameErrors
