Pablo Galindo Salgado <pablog...@gmail.com> added the comment:

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.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue33613>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to