`struct` should be unnecessary here and in other places as I mentioned before. Additionally, I suggest considering using [ClangFormat](http://code.google.com/p/chromium/wiki/ClangFormat) to format your code. If I'm not mistaken, Chromium's (and, consequently, ours) coding style uses `my_pointer* p` instead of `my_pointer *p` (it's weird that the style checker did not complain).