Re: [PATCH 07/15] sequencer: lib'ify read_populate_opts()

2016-08-29 Thread Johannes Schindelin
Hi Junio, On Fri, 26 Aug 2016, Junio C Hamano wrote: > Johannes Schindelin writes: > > > -static void read_populate_opts(struct replay_opts **opts_ptr) > > +static int read_populate_opts(struct replay_opts **opts) > > { > > if

Re: [PATCH 07/15] sequencer: lib'ify read_populate_opts()

2016-08-26 Thread Junio C Hamano
Johannes Schindelin writes: > -static void read_populate_opts(struct replay_opts **opts_ptr) > +static int read_populate_opts(struct replay_opts **opts) > { > if (!file_exists(git_path_opts_file())) > - return; > - if

[PATCH 07/15] sequencer: lib'ify read_populate_opts()

2016-08-23 Thread Johannes Schindelin
To be truly useful, the sequencer should never die() but always return an error. Signed-off-by: Johannes Schindelin --- sequencer.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/sequencer.c b/sequencer.c index 5f6b020..808cd53