Stefan Sperling wrote on Wed, Sep 07, 2011 at 16:46:21 +0200:
> On Wed, Sep 07, 2011 at 05:19:19PM +0300, Daniel Shahaf wrote:
> > This will assert with
> > 
> > % cat $REPOS/db/format
> > 6
> > layout linear
> > %
> 
> Good catch. The assert() is from code I copy-pasted. Will look into it.

You need to decide at what level to check the format number.  For
example, in make_successor_dirs() you check the format number in the
function itself; the other option is to assert() the format number in
the function and check it at the caller, and I believe that's what I did
last time I had to make that call.

Reply via email to