Ignore empty lines in OWNERS files.

owners.py

 # Copyright (c) 2010 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.
 
 """A database of OWNERS files."""
 
 import collections
 import re
 
 
@@ skipping 129 matching lines @@
 
   def _read_owners_in_dir(self, dirpath):
     owners_path = self.os_path.join(self.root, dirpath, 'OWNERS')
     if not self.os_path.exists(owners_path):
       return
 
     lineno = 0
     for line in self.fopen(owners_path):
       lineno += 1
       line = line.strip()
-      if line.startswith('#'):
+      if line.startswith('#') or line == '':

[M-A Ruel, 2011/06/08 21:12:43]

if line.startswith('#') or not line:

         continue
       if line == 'set noparent':
         self.stop_looking.add(dirpath)
         continue
       if line.startswith('set '):
         raise SyntaxErrorInOwnersFile(owners_path, lineno,
             'unknown option: "%s"' % line[4:].strip())
       if self.email_regexp.match(line) or line == EVERYONE:
         self.owned_by.setdefault(line, set()).add(dirpath)
         self.owners_for.setdefault(dirpath, set()).add(line)
         continue
       raise SyntaxErrorInOwnersFile(owners_path, lineno,
             ('line is not a comment, a "set" directive, '
              'or an email address: "%s"' % line))
 
   def _covering_set_of_owners_for(self, files):
     # TODO(dpranke): implement the greedy algorithm for covering sets, and
     # consider returning multiple options in case there are several equally
     # short combinations of owners.
     every_owner = set()
     for f in files:
       dirname = self.os_path.dirname(f)
       while dirname in self.owners_for:
         every_owner |= self.owners_for[dirname]
         if self._stop_looking(dirname):
           break
         dirname = self.os_path.dirname(dirname)
     return every_owner