Message320373
Additionally, as I commented in the PR, apart from the race condition I think the test should be modified so it fails if the warning is raised and not expected:
>The problem is that the test was silently failing when SIGINT was >being delivered as the test does not check that no warnings are >raised. Notice that the presence of a warning means that the process >died and therefore SIGINT did kill the process, which is precisely >what the test checks that does not happen.
>I think in order to check that the test works as intended without the >need to run the suite with -Wall, we need to modify the test. |
|
Date |
User |
Action |
Args |
2018-06-24 14:02:45 | pablogsal | set | recipients:
+ pablogsal, pitrou, vstinner, taleinat, serhiy.storchaka, davin |
2018-06-24 14:02:45 | pablogsal | set | messageid: <1529848965.54.0.56676864532.issue33613@psf.upfronthosting.co.za> |
2018-06-24 14:02:45 | pablogsal | link | issue33613 messages |
2018-06-24 14:02:45 | pablogsal | create | |
|