Initial commit of the Dart Chrome Extension APIs generators

tools/json_schema_compiler/code.py

 # 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.
 
 class Code(object):
   """A convenience object for constructing code.
 
   Logically each object should be a block of code. All methods except |Render|
   and |IsEmpty| return self.
   """
@@ skipping 45 matching lines @@
 
   def Sblock(self, line=''):
     """Starts a code block.
 
     Appends a line of code and then increases the indent level.
     """
     self.Append(line)
     self._indent_level += self._indent_size
     return self
 
+  def SblockBare(self):

[not at google - send to devlin, 2013/01/25 18:14:33]

The nice thing about the other operations (with the exception of Comment)

Append
Cblock
Eblock
Concat

is that they're the same length. When chaining together calls to Code, they line
up vertically, which makes it much easier to scan the code generation.

I would suggest that instead of adding these methods you change Sblock and
Eblock to take None as their default "line" argument, then have

if line is not None:
  self.Append(line)
self._indent_level += self._indent_size
return self

[sashab, 2013/01/29 08:27:13]

Sure - I'm not going to change the default line argument to None, though -
Code.Append() is already being used elsewhere, and it means to add a blank line.

[not at google - send to devlin, 2013/01/29 16:37:08]

Well, if Append didn't add a line given a blank argument then it'd be a no-op. I
wasn't suggesting to change Append, just Sblock and Eblock. And yeah, I think
that would be neater than explicitly writing None.

[sashab, 2013/02/04 05:09:27]

Done.

+    """Starts a code block by increasing the indent, without adding a line.
+    """
+    self._indent_level += self._indent_size
+    return self
+
   def Eblock(self, line=''):
     """Ends a code block by decreasing and then appending a line (or a blank
     line if not given).
     """
     # TODO(calamity): Decide if type checking is necessary
     #if not isinstance(line, basestring):
     #  raise TypeError
     self._indent_level -= self._indent_size
     self.Append(line)
     return self
 
+  def EblockBare(self):
+    """Ends a code block by decreasing the indent, without adding a line.
+    """
+    self._indent_level -= self._indent_size
+    return self
+
   def Comment(self, comment, comment_prefix='// '):
     """Adds the given string as a comment.
 
     Will split the comment if it's too long. Use mainly for variable length
     comments. Otherwise just use code.Append('// ...') for comments.
 
     Unaffected by code.Substitute().
     """
     max_len = self._comment_length - self._indent_level - len(comment_prefix)
     while len(comment) >= max_len:
@@ skipping 34 matching lines @@
     """Renders Code as a string.
     """
     return '\n'.join([l.value for l in self._code])
 
 class Line(object):
   """A line of code.
   """
   def __init__(self, value, substitute=True):
     self.value = value
     self.substitute = substitute