I think Gavin is thinking that complete is a new member variable (when in fact it has just been moved) and therefore was expecting to see an initialization of it in the diff (there is an initialization of it in the constructor).