Code review comment for lp:~ian-clatworthy/bzr/faster-log-file

I just ran a basic benchmark for this branch on the emacs trunk. The performance 'bzr log FILE' takes ~11s for trunk and ~24s for this branch. Maybe something has changed as this branch is quite old or perhaps emacs development model is not flat-ish.
'bzr log' performance is about the same.
Maybe someone familiar with log can comment or mark this as rejected to keep wip queue size down?

[emacs-bzr]% time ~/src/bzr.dev/faster-log-file/bzr --no-plugins log > /dev/null
~/src/bzr.dev/faster-log-file/bzr --no-plugins log > /dev/null 34.97s user 1.76s system 97% cpu 37.860 total
[emacs-bzr]% time ~/src/bzr.dev/faster-log-file/bzr --no-plugins log > /dev/null
~/src/bzr.dev/faster-log-file/bzr --no-plugins log > /dev/null 35.11s user 1.66s system 99% cpu 36.997 total
[emacs-bzr]% time bzr --no-plugins log > /dev/null
bzr --no-plugins log > /dev/null 35.19s user 1.72s system 99% cpu 37.082 total
[emacs-bzr]% time bzr log autogen.sh > /dev/null
bzr log autogen.sh > /dev/null 11.26s user 0.25s system 96% cpu 11.884 total
[emacs-bzr]% time bzr log autogen.sh > /dev/null
bzr log autogen.sh > /dev/null 11.07s user 0.24s system 100% cpu 11.305 total
[emacs-bzr]% time ~/src/bzr.dev/faster-log-file/bzr --no-plugins log autogen.sh > /dev/null
~/src/bzr.dev/faster-log-file/bzr --no-plugins log autogen.sh > /dev/null 23.66s user 0.22s system 99% cpu 24.075 total
[emacs-bzr]% time ~/src/bzr.dev/faster-log-file/bzr --no-plugins log autogen.sh > /dev/null
~/src/bzr.dev/faster-log-file/bzr --no-plugins log autogen.sh > /dev/null 25.39s user 0.28s system 99% cpu 25.756 total
[emacs-bzr]%