From: Anna Zaks <ganna@apple.com>
Date: Wed, 14 Nov 2012 21:32:16 +0000 (+0000)
Subject: [analyzer] Ensure that CmpRuns recursively walks the output directory.
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9821e57c09723dd3e6efad0b2fd9eb29f78753b0;p=platform%2Fupstream%2Fllvm.git

[analyzer] Ensure that CmpRuns recursively walks the output directory.

llvm-svn: 167981
---

diff --git a/clang/utils/analyzer/CmpRuns.py b/clang/utils/analyzer/CmpRuns.py
index 3ca9b2b..9b468bf 100755
--- a/clang/utils/analyzer/CmpRuns.py
+++ b/clang/utils/analyzer/CmpRuns.py
@@ -148,45 +148,46 @@ def loadResultsFromSingleRun(info, deleteEmpty=True):
     path = info.path
     run = AnalysisRun(info)
     
-    for f in os.listdir(path):
-        if (not f.endswith('plist')):
-            continue
-
-        p = os.path.join(path, f)
-        data = plistlib.readPlist(p)
-
-        # Ignore/delete empty reports.
-        if not data['files']:
-            if deleteEmpty == True:
-                os.remove(p)
-            continue
-
-        # Extract the HTML reports, if they exists.
-        if 'HTMLDiagnostics_files' in data['diagnostics'][0]:
-            htmlFiles = []
-            for d in data['diagnostics']:
-                # FIXME: Why is this named files, when does it have multiple
-                # files?
-                assert len(d['HTMLDiagnostics_files']) == 1
-                htmlFiles.append(d.pop('HTMLDiagnostics_files')[0])
-        else:
-            htmlFiles = [None] * len(data['diagnostics'])
-        
-        clang_version = ''
-        if 'clang_version' in data:
-            clang_version = data.pop('clang_version')
-        
-        report = AnalysisReport(run, data.pop('files'), clang_version)
-        diagnostics = [AnalysisDiagnostic(d, report, h) 
-                       for d,h in zip(data.pop('diagnostics'),
-                                      htmlFiles)]
-
-        assert not data
-        
-        report.diagnostics.extend(diagnostics)
-        run.reports.append(report)
-        run.diagnostics.extend(diagnostics)
-
+    for (dirpath, dirnames, filenames) in os.walk(path):
+        for f in filenames:
+            if (not f.endswith('plist')):
+                continue
+    
+            p = os.path.join(dirpath, f)
+            data = plistlib.readPlist(p)
+    
+            # Ignore/delete empty reports.
+            if not data['files']:
+                if deleteEmpty == True:
+                    os.remove(p)
+                continue
+    
+            # Extract the HTML reports, if they exists.
+            if 'HTMLDiagnostics_files' in data['diagnostics'][0]:
+                htmlFiles = []
+                for d in data['diagnostics']:
+                    # FIXME: Why is this named files, when does it have multiple
+                    # files?
+                    assert len(d['HTMLDiagnostics_files']) == 1
+                    htmlFiles.append(d.pop('HTMLDiagnostics_files')[0])
+            else:
+                htmlFiles = [None] * len(data['diagnostics'])
+            
+            clang_version = ''
+            if 'clang_version' in data:
+                clang_version = data.pop('clang_version')
+            
+            report = AnalysisReport(run, data.pop('files'), clang_version)
+            diagnostics = [AnalysisDiagnostic(d, report, h) 
+                           for d,h in zip(data.pop('diagnostics'),
+                                          htmlFiles)]
+    
+            assert not data
+            
+            report.diagnostics.extend(diagnostics)
+            run.reports.append(report)
+            run.diagnostics.extend(diagnostics)
+            
     return run
 
 def cmpAnalysisDiagnostic(d) :