A64: Synchronize with r18084.

tools/presubmit.py

 #!/usr/bin/env python
 #
 # Copyright 2012 the V8 project authors. All rights reserved.
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 #       notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
@@ skipping 173 matching lines @@
 
   def Run(self, path):
     all_files = []
     for file in self.GetPathsToSearch():
       all_files += self.FindFilesIn(join(path, file))
     if not self.ProcessFiles(all_files, path):
       return False
     return True
 
   def IgnoreDir(self, name):
-    return name.startswith('.') or name == 'data' or name == 'sputniktests'
+    return (name.startswith('.') or
+            name in ('data', 'kraken', 'octane', 'sunspider'))
 
   def IgnoreFile(self, name):
     return name.startswith('.')
 
   def FindFilesIn(self, path):
     result = []
     for (root, dirs, files) in os.walk(path):
       for ignored in [x for x in dirs if self.IgnoreDir(x)]:
         dirs.remove(ignored)
       for file in files:
@@ skipping 100 matching lines @@
   def IsRelevant(self, name):
     for ext in SourceProcessor.RELEVANT_EXTENSIONS:
       if name.endswith(ext):
         return True
     return False
 
   def GetPathsToSearch(self):
     return ['.']
 
   def IgnoreDir(self, name):
-    return (super(SourceProcessor, self).IgnoreDir(name)
-              or (name == 'third_party')
-              or (name == 'gyp')
-              or (name == 'out')
-              or (name == 'obj')
-              or (name == 'DerivedSources'))
+    return (super(SourceProcessor, self).IgnoreDir(name) or
+            name in ('third_party', 'gyp', 'out', 'obj', 'DerivedSources'))
 
   IGNORE_COPYRIGHTS = ['cpplint.py',
                        'daemon.py',
                        'earley-boyer.js',
                        'raytrace.js',
                        'crypto.js',
                        'libraries.cc',
                        'libraries-empty.cc',
                        'jsmin.py',
                        'regexp-pcre.js',
@@ skipping 27 matching lines @@
         parts.pop()
       for part in parts:
         line += part.count('\n') + 1
         lines.append(str(line))
       linenumbers = ', '.join(lines)
       if len(lines) > 1:
         print "%s has trailing whitespaces in lines %s." % (name, linenumbers)
       else:
         print "%s has trailing whitespaces in line %s." % (name, linenumbers)
       result = False
+    if not contents.endswith('\n') or contents.endswith('\n\n'):
+      print "%s does not end with a single new line." % name
+      result = False
     # Check two empty lines between declarations.
     if name.endswith(".cc"):
       line = 0
       lines = []
       parts = contents.split('\n')
       while line < len(parts) - 2:
         if self.EndOfDeclaration(parts[line]):
           if self.StartOfDeclaration(parts[line + 1]):
             lines.append(str(line + 1))
             line += 1
@@ skipping 48 matching lines @@
         "two empty lines between declarations check..."
   success = SourceProcessor().Run(workspace) and success
   if success:
     return 0
   else:
     return 1
 
 
 if __name__ == '__main__':
   sys.exit(Main())