Code review comment for lp:~jameinel/bzr/1.16-chkmap-updates

Revision history for this message
Ian Clatworthy (ian-clatworthy) wrote :

This looks pretty good. I want to take another look when I'm a bit more awake though before approving it.

One small thing: you changed one instance of "type(node) == tuple" to "type(node) is tuple" but you missed 2 other spots in that routine. I'll finish reviewing it tomorrow.

« Back to merge proposal