[Git][reproducible-builds/diffoscope][master] Fix slightly misplaced .debug("Loading diff from stdin"). (Closes: !40)

[Chris Lamb]

Chris Lamb pushed to branch master at Reproducible Builds / diffoscope


Commits:
c150ae8e by Marc Herbert at 2020-01-10T16:15:21+00:00
Fix slightly misplaced .debug("Loading diff from stdin"). (Closes: !40)

Spotted by chance while looking at unrelated performance issue #84

This line was added by commit 509509e30287

Signed-off-by: Chris Lamb <lamby at debian.org>

- - - - -


1 changed file:

- diffoscope/main.py


Changes:

=====================================
diffoscope/main.py
=====================================
@@ -663,10 +663,11 @@ def run_diffoscope(parsed_args):
     set_locale()
     path1, path2 = parsed_args.path1, parsed_args.path2
     if path2 is None:
-        logger.debug("Loading diff from stdin")
         if path1 == '-':
+            logger.debug("Loading diff from stdin")
             difference = load_diff(sys.stdin, "stdin")
         else:
+            logger.debug("Loading diff from %s", path1)
             try:
                 difference = load_diff_from_path(path1)
             except json.JSONDecodeError:



View it on GitLab: https://salsa.debian.org/reproducible-builds/diffoscope/commit/c150ae8e62f3779ae7617c814e8e7203123a67ad

-- 
View it on GitLab: https://salsa.debian.org/reproducible-builds/diffoscope/commit/c150ae8e62f3779ae7617c814e8e7203123a67ad
You're receiving this email because of your account on salsa.debian.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.reproducible-builds.org/pipermail/rb-commits/attachments/20200110/02bddefa/attachment.htm>