* Paul J. Reder wrote: > With the other changes, the way I read the patch was as: > > if (ctx->state == PRE_HEAD) { > if (!empty) { > pass > /* reset deleted from here */ > } > reset /* reset added here */ > }
yep... I meant, one could equally write the patch as: if (ctx->state == PRE_HEAD) { if (!empty) { pass reset } else { reset } } where the else path means if(empty). I'm just confused, why not to reset in that case. nd