[diffoscope] 04/05: Profile the "has_same_content_as" methods.

Chris Lamb chris at chris-lamb.co.uk
Fri Dec 23 12:46:22 CET 2016


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

lamby pushed a commit to branch master
in repository diffoscope.

commit 4759c9129c577e8d686e320baa880b58d0ff6956
Author: Chris Lamb <lamby at debian.org>
Date:   Fri Dec 23 11:37:48 2016 +0000

    Profile the "has_same_content_as" methods.
    
    Signed-off-by: Chris Lamb <lamby at debian.org>
---
 diffoscope/comparators/__init__.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/diffoscope/comparators/__init__.py b/diffoscope/comparators/__init__.py
index 93d839d..7113db3 100644
--- a/diffoscope/comparators/__init__.py
+++ b/diffoscope/comparators/__init__.py
@@ -27,6 +27,7 @@ import importlib
 
 from diffoscope import logger, tool_required
 from diffoscope.config import Config
+from diffoscope.profiling import profile
 from diffoscope.difference import Difference
 
 from .binary import NonExistingFile
@@ -133,9 +134,10 @@ def compare_root_paths(path1, path2):
 
 def compare_files(file1, file2, source=None):
     logger.debug("Comparing files %s and %s", file1, file2)
-    if file1.has_same_content_as(file2):
-        logger.debug("has_same_content_as returned True; skipping further comparisons")
-        return None
+    with profile('has_same_content_as', file1):
+        if file1.has_same_content_as(file2):
+            logger.debug("has_same_content_as returned True; skipping further comparisons")
+            return None
     specialize(file1)
     specialize(file2)
     if isinstance(file1, NonExistingFile):

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


More information about the diffoscope mailing list