From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Wed, 12 Oct 2016 12:06:09 GMT
Subject: Re: [PATCH v3 05/25] sequencer: eventually release memory allocated for the option values
Message-ID: <alpine.DEB.2.20.1610121405410.3492@virtualbox>
In-Reply-To: <xmqq4m4ic0gw.fsf@gitster.mtv.corp.google.com>

Hi Junio,

On Tue, 11 Oct 2016, Junio C Hamano wrote:

> The only reason why the OPT_STRDUP appeared convenient was because
> options[] element happened to use a field in the structure directly.
> The patch under discussion does an equivalent of
> 
>     app.x_field = xstrdup_or_null(opt_x);

Oh, that xstrdup_or_null() function slipped by me. My local patches use it
now.

Ciao,
Dscho