Title: [285522] trunk/Tools
Revision
285522
Author
stephan.sz...@sony.com
Date
2021-11-09 12:25:57 -0800 (Tue, 09 Nov 2021)

Log Message

[Windows] Non-find based status file finding in run-jsc-stress-tests needs to release status files
https://bugs.webkit.org/show_bug.cgi?id=232851

Reviewed by Don Olmstead.

Change the reading of status files in the non-find search
to properly scope the file access to release the status
files.

* Scripts/run-jsc-stress-tests:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (285521 => 285522)


--- trunk/Tools/ChangeLog	2021-11-09 20:04:19 UTC (rev 285521)
+++ trunk/Tools/ChangeLog	2021-11-09 20:25:57 UTC (rev 285522)
@@ -1,3 +1,16 @@
+2021-11-09  Stephan Szabo  <stephan.sz...@sony.com>
+
+        [Windows] Non-find based status file finding in run-jsc-stress-tests needs to release status files
+        https://bugs.webkit.org/show_bug.cgi?id=232851
+
+        Reviewed by Don Olmstead.
+
+        Change the reading of status files in the non-find search
+        to properly scope the file access to release the status
+        files.
+
+        * Scripts/run-jsc-stress-tests:
+
 2021-11-09  Chris Dumez  <cdu...@apple.com>
 
         New spec: Block external protocol handler in sandboxed frames

Modified: trunk/Tools/Scripts/run-jsc-stress-tests (285521 => 285522)


--- trunk/Tools/Scripts/run-jsc-stress-tests	2021-11-09 20:04:19 UTC (rev 285521)
+++ trunk/Tools/Scripts/run-jsc-stress-tests	2021-11-09 20:25:57 UTC (rev 285522)
@@ -2367,8 +2367,10 @@
         Dir.chdir($runnerDir) {
             Dir.glob("#{STATUS_FILE_PREFIX}*").each do |name|
                 if File.size(name) > 0
-                    line = File.open(name).first
-                    processStatusLine(map, "./#{name} #{line}")
+                    File.open(name, 'r') { |f|
+                        line = f.first
+                        processStatusLine(map, "./#{name} #{line}")
+                    }
                 end
             end
         }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to