Allow longer branch names to be center justified with git cl status.

git_cl.py

 #!/usr/bin/env python
 # 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.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld."""
 
 import logging
@@ skipping 781 matching lines @@
   parser.add_option('--field',
                     help='print only specific field (desc|id|patch|url)')
   (options, args) = parser.parse_args(args)
 
   # TODO: maybe make show_branches a flag if necessary.
   show_branches = not options.field
 
   if show_branches:
     branches = RunGit(['for-each-ref', '--format=%(refname)', 'refs/heads'])
     if branches:
-      print 'Branches associated with reviews:'
+      alignment = 5

[M-A Ruel, 2012/04/02 19:56:25]

You can code it even shorter with:

print 'Branches associated with reviews:'
changes = (Changelist(branchref=b) for b in branches.splitlines())
branches = dict((cl.GetBranch(), cl.GetIssue()) for cl in changes)
alignment = max(5, max(len(b) for b in branches))
for branch in sorted(branches):
  print "  %*s: %s" % (alignment, branch, branches[branch])

+      branch_info = []
       for branch in sorted(branches.splitlines()):
         cl = Changelist(branchref=branch)
-        print "  %10s: %s" % (cl.GetBranch(), cl.GetIssue())
+        alignment = max(alignment, len(cl.GetBranch()))
+        branch_info.append([cl.GetBranch(), cl.GetIssue()])
+
+      print 'Branches associated with reviews:'
+      for info in branch_info:
+        print "  %*s: %s" % (alignment, info[0], info[1])
 
   cl = Changelist()
   if options.field:
     if options.field.startswith('desc'):
       print cl.GetDescription()
     elif options.field == 'id':
       issueid = cl.GetIssue()
       if issueid:
         print issueid
     elif options.field == 'patch':
@@ skipping 703 matching lines @@
              'and retry or visit go/isgaeup.\n%s') % (e.code, str(e)))
 
   # Not a known command. Default to help.
   GenUsage(parser, 'help')
   return CMDhelp(parser, argv)
 
 
 if __name__ == '__main__':
   fix_encoding.fix_encoding()
   sys.exit(main(sys.argv[1:]))