Hi Liam, On Mon, 4 Dec 2017, Johannes Schindelin wrote: > On Sun, 3 Dec 2017, Liam Beguin wrote: > > > diff --git a/sequencer.h b/sequencer.h > > index 4e444e3bf1c4..3bb6b0658192 100644 > > --- a/sequencer.h > > +++ b/sequencer.h > > @@ -45,10 +45,12 @@ int sequencer_continue(struct replay_opts *opts); > > int sequencer_rollback(struct replay_opts *opts); > > int sequencer_remove_state(struct replay_opts *opts); > > > > -int sequencer_make_script(int keep_empty, FILE *out, > > - int argc, const char **argv); > > +#define TODO_LIST_KEEP_EMPTY (1U << 0) > > +#define TODO_LIST_SHORTED_IDS (1U << 1) > > Maybe SHORTEN_IDs? And either revert back to transform_todo_ids() or use > SHORTEN_INSNS... > > Maybe also TRANSFORM_TODO_LIST_* and maybe move the #define's above the > transform_todo_ids() function, i.e. one line further down? > > > +int sequencer_make_script(FILE *out, int argc, const char **argv, > > + unsigned flags); Ah, I just realized that make_script() already takes `flags`. Sorry for the noise, Johannes
