Hi Junio, On Wed, 12 Oct 2016, Junio C Hamano wrote: > Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > > > 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. > > It has slipped many people ;-) Thanks, I feel better now ;-) Ciao, Dscho