Merge with stable.
authorAugie Fackler <raf@durin42.com>
Wed, 06 Nov 2013 12:53:39 -0500
changeset 19985 aa80446aacc3
parent 19983 e1317d3e59e1 (diff)
parent 19984 7d5e7799a29f (current diff)
child 19988 8dc6f855f43d
Merge with stable.
--- a/contrib/check-code.py	Mon Nov 04 19:59:00 2013 -0800
+++ b/contrib/check-code.py	Wed Nov 06 12:53:39 2013 -0500
@@ -247,7 +247,7 @@
     (r'[\s\(](open|file)\([^)]*\)\.read\(',
      "use util.readfile() instead"),
     (r'[\s\(](open|file)\([^)]*\)\.write\(',
-     "use util.readfile() instead"),
+     "use util.writefile() instead"),
     (r'^[\s\(]*(open(er)?|file)\([^)]*\)',
      "always assign an opened file to a variable, and close it afterwards"),
     (r'[\s\(](open|file)\([^)]*\)\.',
@@ -456,6 +456,8 @@
                 if prelines is None:
                     prelines = pre.splitlines()
                     postlines = post.splitlines(True)
+                    if i >= nerrs:
+                        msg = "warning: " + msg
 
                 start = m.start()
                 while n < len(postlines):
@@ -482,8 +484,6 @@
                         bl, bu, br = blamecache[n]
                         if bl == l:
                             bd = '%s@%s' % (bu, br)
-                if i >= nerrs:
-                    msg = "warning: " + msg
                 errors.append((f, lineno and n + 1, l, msg, bd))
                 result = False