bpo-39916: Use os.scandir() as context manager in Path.glob(). (GH-18880)

miss-islington · serhiy-storchaka · web-flow · commit b1b1d5ff1110 · 2020-03-11T10:00:09.000-07:00
(cherry picked from commit 704e206) Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
diff --git a/Lib/pathlib.py b/Lib/pathlib.py
@@ -522,7 +522,8 @@ def __init__(self, pat, child_parts, flavour):
 
     def _select_from(self, parent_path, is_dir, exists, scandir):
         try:
-            entries = list(scandir(parent_path))
+            with scandir(parent_path) as scandir_it:
+                entries = list(scandir_it)
             for entry in entries:
                 if self.dironly:
                     try:
@@ -552,7 +553,8 @@ def __init__(self, pat, child_parts, flavour):
     def _iterate_directories(self, parent_path, is_dir, scandir):
         yield parent_path
         try:
-            entries = list(scandir(parent_path))
+            with scandir(parent_path) as scandir_it:
+                entries = list(scandir_it)
             for entry in entries:
                 entry_is_dir = False
                 try:
diff --git a/Misc/NEWS.d/next/Library/2020-03-09-18-56-27.bpo-39916.BHHyp3.rst b/Misc/NEWS.d/next/Library/2020-03-09-18-56-27.bpo-39916.BHHyp3.rst
@@ -0,0 +1,2 @@
+More reliable use of ``os.scandir()`` in ``Path.glob()``. It no longer emits
+a ResourceWarning when interrupted.

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	+More reliable use of ``os.scandir()`` in ``Path.glob()``. It no longer emits
	`2`	`+a ResourceWarning when interrupted.`