Mojo: Reformat parser_unittest.py.

mojo/public/tools/bindings/pylib/mojom/parse/ast.py

 # Copyright 2014 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.
 
 """Node objects for the AST for a Mojo IDL file."""
 
 # Note: For convenience of testing, you probably want to define __eq__() methods
 # for all node types; it's okay to be slightly lax (e.g., not compare filename
 # and lineno).
 
 
 class BaseNode(object):
   def __init__(self, filename=None, lineno=None):
     self.filename = filename
     self.lineno = lineno
 
 
 class Ordinal(BaseNode):
   """Represents an ordinal value labeling, e.g., a struct field."""
+

[jamesr, 2014/07/02 17:36:29]

is this what the pep8 formatter would do?

[viettrungluu, 2014/07/02 17:59:04]

I think so. It'd probably also put a blank line before BaseNode's __init__.
Maybe I'll also add a docstring for BaseNode.

   def __init__(self, value, **kwargs):
     BaseNode.__init__(self, **kwargs)
     self.value = value
 
   def __eq__(self, other):
     return self.value == other.value
 
+
 class Parameter(BaseNode):
   """Represents a method request or response parameter."""
+
   def __init__(self, typename, name, ordinal, **kwargs):
     assert isinstance(ordinal, Ordinal)
     BaseNode.__init__(self, **kwargs)
     self.typename = typename
     self.name = name
     self.ordinal = ordinal
 
   def __eq__(self, other):
     return self.typename == other.typename and \
            self.name == other.name and \
            self.ordinal == other.ordinal