Re: [PATCH 04/22] sequencer: future-proof remove_sequencer_state()

2016-08-29 Thread Johannes Schindelin
Hi Dennis, On Mon, 29 Aug 2016, Johannes Schindelin wrote: > On Mon, 29 Aug 2016, Dennis Kaarsemaker wrote: > > > On ma, 2016-08-29 at 10:04 +0200, Johannes Schindelin wrote: > > > > > +   if (read_and_refresh_cache(opts)) > > > +   return -1; > > > + > > > > This doesn't seem

Re: [PATCH 04/22] sequencer: future-proof remove_sequencer_state()

2016-08-29 Thread Johannes Schindelin
Hi Dennis, On Mon, 29 Aug 2016, Dennis Kaarsemaker wrote: > On ma, 2016-08-29 at 10:04 +0200, Johannes Schindelin wrote: > > > +   if (read_and_refresh_cache(opts)) > > +   return -1; > > + > > This doesn't seem to be related to the get_dir changes? Good eyes. Let me investiga

Re: [PATCH 04/22] sequencer: future-proof remove_sequencer_state()

2016-08-29 Thread Dennis Kaarsemaker
On ma, 2016-08-29 at 10:04 +0200, Johannes Schindelin wrote: > +   if (read_and_refresh_cache(opts)) > +   return -1; > + This doesn't seem to be related to the get_dir changes? D.