Presubmit tests: allow 100columns limit for .java files.

presubmit_support.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Enables directory-specific presubmit checks to run at upload and/or commit.
 """
 
 __version__ = '1.6.1'
 
@@ skipping 436 matching lines @@
   def LocalPath(self):
     """Returns the path of this file on the local disk relative to client root.
     """
     return normpath(self._path)
 
   def AbsoluteLocalPath(self):
     """Returns the absolute path of this file on the local disk.
     """
     return os.path.abspath(os.path.join(self._local_root, self.LocalPath()))
 
+  def GetFileExt(self):

[M-A Ruel, 2012/02/16 22:35:43]

I'd rather not add _yet another member_. Since it's used only at one place, just
to a endswith() in the canned checks or rsplit('.', 1)[-1], it's fine enough.

[bulach, 2012/02/17 01:25:15]

sounds reasonable. removed..

+    """Returns the extension of this file."""
+    return os.path.splitext(self._path)[1]
+    
   def IsDirectory(self):
     """Returns true if this object is a directory."""
     if self._is_directory is None:
       path = self.AbsoluteLocalPath()
       self._is_directory = (os.path.exists(path) and
                             os.path.isdir(path))
     return self._is_directory
 
   def Action(self):
     """Returns the action on this opened file, e.g. A, M, D, etc."""
@@ skipping 798 matching lines @@
   except PresubmitFailure, e:
     print >> sys.stderr, e
     print >> sys.stderr, 'Maybe your depot_tools is out of date?'
     print >> sys.stderr, 'If all fails, contact maruel@'
     return 2
 
 
 if __name__ == '__main__':
   fix_encoding.fix_encoding()
   sys.exit(Main(None))