Jonathan Tan <jonathantanmy@xxxxxxxxxx> writes: > But in v2: > > if (!haves_added || *in_vain >= MAX_IN_VAIN) { > > ("haves_added" is irrelevant to this discussion. It is another > termination condition - when we have run out of "have"s to send.) > > So there is no check that "continue" was sent. We probably should change > v2 to match v0. That sounds like a good change. Thanks.