Johannes Schindelin <johannes.schindelin@xxxxxx> writes: > -static int read_populate_opts(struct replay_opts **opts) > +static int read_populate_opts(struct replay_opts *opts) > { > if (!file_exists(git_path_opts_file())) > return 0; > @@ -823,7 +823,7 @@ static int read_populate_opts(struct replay_opts **opts) > * about this case, though, because we wrote that file ourselves, so we > * are pretty certain that it is syntactically correct. > */ > - if (git_config_from_file(populate_opts_cb, git_path_opts_file(), *opts) < 0) > + if (git_config_from_file(populate_opts_cb, git_path_opts_file(), opts) < 0) > return error(_("Malformed options sheet: %s"), > git_path_opts_file()); > return 0; > @@ -1054,7 +1054,7 @@ static int sequencer_continue(struct replay_opts *opts) > > if (!file_exists(git_path_todo_file())) > return continue_single_pick(); > - if (read_populate_opts(&opts) || > + if (read_populate_opts(opts) || > read_populate_todo(&todo_list, opts)) > return -1; Good!