[diffoscope] 01/01: Remove pointless use of a thread

Ximin Luo infinity0 at debian.org
Thu Feb 9 21:03:51 CET 2017


This is an automated email from the git hooks/post-receive script.

infinity0 pushed a commit to branch master
in repository diffoscope.

commit 6ec3e00b7f1ab3835b320bf9b46bf703b3e1011c
Author: Ximin Luo <infinity0 at debian.org>
Date:   Thu Feb 9 21:03:33 2017 +0100

    Remove pointless use of a thread
    
    This is a leftover from continued refactorings starting all the way back in
    debbindiff, when this part actually did run two threads - see `git log -p --
    debbindiff/difference.py` for details.
    
    However the curent form (start-join) is just pointless so get rid of it.
---
 diffoscope/diff.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/diffoscope/diff.py b/diffoscope/diff.py
index de2c416..5ec2ade 100644
--- a/diffoscope/diff.py
+++ b/diffoscope/diff.py
@@ -172,10 +172,7 @@ def run_diff(fifo1, fifo2, end_nl_q1, end_nl_q2):
     p.stdin.close()
 
     parser = DiffParser(p.stdout, end_nl_q1, end_nl_q2)
-    t_read = threading.Thread(target=parser.parse)
-    t_read.daemon = True
-    t_read.start()
-    t_read.join()
+    parser.parse()
     p.wait()
 
     logger.debug(

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/reproducible/diffoscope.git


More information about the diffoscope mailing list