> From: Felipe Contreras [mailto:felipe.contre...@gmail.com]
> Sent: Wednesday, May 29, 2013 12:52 PM
> To: Joachim Schmitz
> Cc: email@example.com
> Subject: Re: [PATCH 1/2] sequencer: trivial fix
> On Wed, May 29, 2013 at 4:58 AM, Joachim Schmitz
> <j...@schmitz-digital.de> wrote:
> > Felipe Contreras wrote:
> >> Junio C Hamano wrote:
> >>> It probably is better to fold this patch into the other one when it
> >>> is rerolled to correct the option name gotcha "on the tin".
> >> Why? This patch is standalone and fixes an issue that is independent
> >> of the other patch. Why squash two patches that do *two* different
> >> things?
> >> Anyway, I'll happily drop this patch if you want this memory leak to
> >> remain. But then I'll do the same in the other patch.
> >> This mantra of avodiing 'goto' is not helping anybody.
> > adding 5 letters (to change the next "if" into an "else if") versus your
> > addition of several lines and some 15 additional letters (ignoring the
> > whitsspace) is IMHO enough to see what is better?
> This has nothing to do with what Junio said.
Well, it has, but you had snipped it. But replied to the goto issue regardless
> This is better done without "goto" in general.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html