Code review comment for lp:~ian-clatworthy/bzr/faster-dirstate-saving

I was happy to take this out of the queue while we focussed on 2.0. Right now though, I'm pretty confused here as to why this is rejected. Can someone do a proper review please and tell me exactly what's wrong?

To my understanding, I addressed the first round of comments. If I need to rename a variable or method, that's approve-after-tweak, not reject surely.