Merge branch 'master' into new_vfs_library

1 files changed, 66 insertions, 33 deletions

diff --git a/massage_messages.py b/massage_messages.py
index 249068ba..d79f4cea 100644
--- a/massage_messages.py
+++ b/massage_messages.py
@@ -1,3 +1,5 @@
+from collections import defaultdict
+
 import fileinput
 import re
 import subprocess
@@ -28,14 +30,19 @@ class QWidget;
 namespace Ui { class AboutDialog; }  // lines 31-31
 ---
 """
+
 removing = None
+includes = dict()
 
-includes = dict
+adding = None
+added = dict()
 
-foundline = 0
+foundline = None
 
 errors = False
 
+messages = defaultdict(list)
+
 def process_next_line(line, outfile):
     """ Read a line of output/error from include-what-you use
         Turn clang errors into a form QT creator recognises
@@ -45,34 +52,53 @@ def process_next_line(line, outfile):
     global includes
     global foundline
     global errors
+    global added
+    global adding
     line = line.rstrip()
     print >> outfile, line
-    if removing:
-        if line == '':
-            removing = None
-            print
-            return
+
+    if line.endswith(' should remove these lines:'):
+        adding = None
+        removing = (line.split(' '))[0]
+    elif line.endswith(' should add these lines:'):
+        removing = None
+        adding = (line.split(' '))[0]
+    elif line == '' or line.startswith(' the full include-list'):
+        adding = None
+        removing = None
+    elif adding:
+        m = re.match(r'.*class (.*);', line)
+        if m:
+            added[m.group(1)] = (adding, line)
         else:
-            # Really we should stash these so that if we get a 'class xxx' in
-            # the add lines we can print it here. also we could do the case
-            # fixing.
-            m = re.match(r'- #include [<"](.*)[">] +// lines (.*)-', line)
-            if m:
-                # If there is an added line with the same class, print it here
-                print '%s(%s) : warning I0001: Unnecessary include of %s' %\
-                                              (removing, m.group(2), m.group(1))
-                foundline = m.group(1)
+            added[line] = (adding, line)
+    elif removing:
+        # Really we should stash these so that if we get a 'class xxx' in
+        # the add lines we can print it here. also we could do the case
+        # fixing.
+        m = re.match(r'- #include [<"](.*)[">] +// lines (.*)-', line)
+        if m:
+            foundline = m.group(2)
+            if m.group(1) in added:
+                messages[removing].append(
+                    '%s(%s) : warning I0004: Replace include of %s with '
+                        'forward reference %s' % (
+                        removing, m.group(2), m.group(1), added[m.group(1)][1]))
+                del added[m.group(1)]
             else:
-                m = re.match(r'- (.*) +// lines (.*)-', line)
-                if m:
-                    print '%s(%s) : warning I0002: '\
-                                              'Unnecessary forward ref of %s' %\
-                                              (removing, m.group(2), m.group(1))
-                    foundline = m.group(1)
-                else:
-                    print '********* I got confused **********'
+                messages[removing].append(
+                    '%s(%s) : warning I0001: Unnecessary include of %s' % (
+                                              removing, m.group(2), m.group(1)))
 
-    if line.startswith('In file included from'):
+        else:
+            m = re.match(r'- (.*) +// lines (.*)-', line)
+            if m:
+                messages[removing].append(
+                    '%s(%s) : warning I0002: Unnecessary forward ref of %s' % (
+                                              removing, m.group(2), m.group(1)))
+            else:
+                print '********* I got confused **********'
+    elif line.startswith('In file included from'):
         line = re.sub(r'^(In file included from)(.*):(\d+):',
                       r'        \2(\3) : \1 here',
                       line)
@@ -88,14 +114,6 @@ def process_next_line(line, outfile):
             errors = True
 
     print line
-    if line.endswith(' should remove these lines:'):
-        removing = (line.split(' '))[0]
-    elif line.endswith(' should add these lines:'):
-        adding = (line.split(' '))[0]
-
-# also process the other lines
-
-    # added lines should come after the first entry with a line number.
 
 outfile = open(sys.argv[1], 'w')
 process = subprocess.Popen(sys.argv[2:],
@@ -107,6 +125,21 @@ while True:
     if output:
         process_next_line(output, outfile)
 
+# A note: We should probably do some work as we use the source code line for
+# messages in include files...
+
+if foundline is None:
+    foundline = '1'
+
+for add in added:
+    messages[added[add][0]].append(
+        '%s(%s) : warning I0003: Need to include %s' % (
+                                       added[add][0], foundline, added[add][1]))
+
+for file in sorted(messages.keys(), reverse = True):
+    for line in messages[file]:
+        print line
+
 rc = process.poll()
 # The return code you get appears to be more to do with the amount of output
 # generated than any real error, so instead we should error if any ': error:'