| Summary: | [ews] Ensure file handle is not leaked in loadConfig.py | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||
| Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | aakash_jain, ap, dewei_zhu, jbedard, ryanhaddad, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | Other | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=225360 | ||||||
| Attachments: |
|
||||||
|
Description
Aakash Jain
2021-08-06 09:42:05 PDT
Created attachment 435074 [details]
Patch
Comment on attachment 435074 [details]
Patch
The change is clearly an improvement, but I'm curious if we actually observed the problem in practice (i.e. observed that garbage collection didn't do it quickly).
(In reply to Alexey Proskuryakov from comment #2) > Comment on attachment 435074 [details] > I'm curious if we actually observed the problem in practice. Not really. The motivation for this change was that when the unit-test fail for something, they also print following error about this closed file. This sometimes makes the unit-test failure message ugly and confusing (e.g. https://ews-build.webkit.org/#/builders/20/builds/53092/steps/12/logs/stdio). Stderr: Exception ignored in: <_io.FileIO name='Tools/CISupport/ews-build/config.json' mode='rb' closefd=True> ResourceWarning: unclosed file <_io.TextIOWrapper name='Tools/CISupport/ews-build/config.json' mode='r' encoding='UTF-8'> Committed r280759 (240344@main): <https://commits.webkit.org/240344@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 435074 [details]. |