Tweak the PPAPI IDL generator to wrap the generated-from comment if necessary.

ppapi/generators/idl_c_header.py

 #!/usr/bin/python
 #
 # Copyright (c) 2011 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.
 
 """ Generator for C style prototypes and definitions """
 
 import glob
 import os
@@ skipping 112 matching lines @@
     release = releases[0]
     def_guard = GetOutFileName(filenode, relpath=gpath)
     def_guard = def_guard.replace(os.sep,'_').replace('.','_').upper() + '_'
 
     cright_node = filenode.GetChildren()[0]
     assert(cright_node.IsA('Copyright'))
     fileinfo = filenode.GetChildren()[1]
     assert(fileinfo.IsA('Comment'))
 
     out.Write('%s\n' % cgen.Copyright(cright_node))
-    out.Write('/* From %s modified %s. */\n\n'% (
-        filenode.GetProperty('NAME').replace(os.sep,'/'),
-        filenode.GetProperty('DATETIME')))
+
+    # Wrap the From ... modified ... comment if it would be >80 characters.
+    from_text = '/* From %s' % (
+        filenode.GetProperty('NAME').replace(os.sep,'/'))
+    modified_text = 'modified %s. */' % (
+        filenode.GetProperty('DATETIME'))
+    if len(from_text) + len(modified_text) < 80:
+      out.Write('%s %s\n\n' % (from_text, modified_text))
+    else:
+      out.Write('%s,\n *   %s\n\n' % (from_text, modified_text))

[dmichael (off chromium), 2012/01/04 16:42:25]

So this won't handle if the first part (the file name) is already too long. But
I guess that's fine.
I think when C comments go multi-line elsewhere, we stick the closing "*/" on a
new line. I like the look of that better, if you don't mind

+
     out.Write('#ifndef %s\n#define %s\n\n' % (def_guard, def_guard))
     # Generate set of includes
 
     deps = set()
     for release in releases:
       deps |= filenode.GetDeps(release)
 
     includes = set([])
     for dep in deps:
       depfile = dep.GetProperty('FILE')
@@ skipping 71 matching lines @@
     print "Golden file for M13-M15 failed."
     failed =1
   else:
     print "Golden file for M13-M15 passed."
 
   return failed
 
 if __name__ == '__main__':
   retval = Main(sys.argv[1:])
   sys.exit(retval)