Re: [PATCH] sequencer: fix missing newline

From: Jeff King <peff@peff.net>
Date: Mar 23, 2017, 18:22 UTC
Message-ID: <20170323182221.i3noikxpmrn4ymi4@sigill.intra.peff.net>
In-Reply-To: <20170323174716.GA111250@google.com>

On Thu, Mar 23, 2017 at 10:47:16AM -0700, Brandon Williams wrote:

Show 17 quoted lines

> On 03/23, Johannes Schindelin wrote:
> > Hi Brandon,
> > 
> > On Thu, 23 Mar 2017, Brandon Williams wrote:
> > 
> > > When using rebase --interactive where one of the lines is marked as
> > > 'edit' this is the resulting output:
> > > 
> > >     Stopped at ec3b9c4...  stuffYou can amend the commit now, with
> > 
> > Ugh, I should have caught this. The warning() call implicitly adds a
> > newline, the fprintf() doesn't.
> > 
> > Patch is obviously good.
> 
> All good.  If we're keeping score for finding each others bugs, I still
> think you're winning :)

It's my bug, actually. Brown paper bag, anyone?

-Peff