From: Eric Sunshine <sunshine@sunshineco.com>
Date: Fri, 10 Jun 2016 23:42:12 GMT
Subject: Re: [PATCH 00/27] nd/shallow-deepen updates
Message-ID: <CAPig+cRfUjpku1Abf=GxrbFeS4txwr=yZsJ7EKo-aWmxYog6Xw@mail.gmail.com>
In-Reply-To: <20160610122714.3341-1-pclouds@gmail.com>

On Fri, Jun 10, 2016 at 8:26 AM, Nguyễn Thái Ngọc Duy <pclouds@gmail.com> wrote:
> This contains cleanups after Eric's comments (all good points,
> thanks!). Changed patches have Junio's s-o-b line removed, so it's
> easy to see which is changed and which is not. 09/27 is a new one,
> split out of 10/27.

Thanks, the interdiff looks sensible and I think it covers all the
points raised by my review of the previous round.

I agree with Junio that moving the sigchain_pop() into the error
handling code-path, if possible, would be a nice improvement.