right, I was imprecise. It's an in-out parameter since it's modified by the method. To make it clear to the caller it should be a pointer instead of a reference (there's a lot of other places like this, but this one is new and self-contained). Rationale: http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Reference_Arguments