[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