[Python-checkins] r75526 - python/branches/py3k/Lib/test/regrtest.py

Neal Norwitz nnorwitz at gmail.com
Tue Oct 20 06:44:28 CEST 2009


On Mon, Oct 19, 2009 at 9:54 AM, r.david.murray
<python-checkins at python.org> wrote:
> Author: r.david.murray
> Date: Mon Oct 19 18:53:55 2009
> New Revision: 75526
>
> Log:
> Since all regrtest -j output is now done in the main thread we are no
> longer using the print lock, so remove it.
>
>
>
> Modified:
>   python/branches/py3k/Lib/test/regrtest.py
>
> Modified: python/branches/py3k/Lib/test/regrtest.py
> ==============================================================================
> --- python/branches/py3k/Lib/test/regrtest.py   (original)
> +++ python/branches/py3k/Lib/test/regrtest.py   Mon Oct 19 18:53:55 2009
> @@ -440,13 +440,12 @@
>                 resource_denieds.append(test)
>
>     if use_mp:
> -        from threading import Thread, Lock
> +        from threading import Thread
>         from queue import Queue, Empty
>         from subprocess import Popen, PIPE, STDOUT
>         from collections import deque
>         # TextIOWrapper is not entirely thread-safe now,
>         # it can produce duplicate output when printing from several threads.
> -        print_lock = Lock()
>         debug_output_pat = re.compile(r"\[\d+ refs\]$")
>         pending = deque()
>         output = Queue()

Should the comment about TextIOWrapper also be removed?

n


More information about the Python-checkins mailing list