Merge pull request #737 from samuela/samuela/unicode-error

Catch/report `UnicodeDecodeError`s in `ReadFile`

Closes #736

CHANGELOG

@@ -24,6 +24,7 @@
 ### Changed
 - Set `INDENT_DICTIONARY_VALUE` for Google style.
 - Set `JOIN_MULTIPLE_LINES = False` for Google style.
+- Catch and report `UnicodeDecodeError` exceptions.
 ### Fixed
 - `BLANK_LINE_BEFORE_NESTED_CLASS_OR_DEF=False` wasn't honored because the
   number of newlines was erroneously calculated beforehand.

yapf/yapflib/yapf_api.py

@@ -202,6 +202,12 @@ def ReadFile(filename, logger=None):
     if logger:
       logger(err)
     raise
+  except UnicodeDecodeError as err:  # pragma: no cover
+    if logger:
+      logger('Could not parse %s! Consider excluding this file with --exclude.',
+             filename)
+      logger(err)
+    raise
 
 
 def _SplitSemicolons(uwlines):