[PATCH 3 of 4] Add option to heads to show only heads for current branch

[Matt Mackall]

On Mon, Jun 18, 2007 at 08:06:46PM -0700, Eric Hopper wrote:
> > > +                if branch != branchrev:
> > > +                    ui.warn(_("No changes on branch %s (the branch of revision"\
> > > +                                  " %s) are reachable from revision %s\n") % \
> > > +                                (branch, branchrev, opts['rev']))
> > > +                else:
> > > +                    ui.warn(_("No changes on branch %s are reachable "\
> > > +                                  "from revision %s\n") % \
> > > +                                (branch, opts['rev']))
> > 
> > Indentation, and you don't need '\' here.
> 
> The question now in 'whitespace matters' in Python is "Where?".  It's
> sometimes hard to tell.

It should line up at the same level as the thing on the prior line at
that nesting level. Like this:

func(foo,
     (bar and
      baz),
     quux)

And that doesn't need any backslashes as it's all inside of parens.

Since we're on the subject, most of our warning messages are
lowercase. And your first message is a bit awkward. Perhaps:

 no changes on branch %s containing %s are reachable from %s

-- 
Mathematics is the supreme nostalgia of our time.