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