On 26 April 2013 19:47, David Fang <fang at csl.cornell.edu> wrote: > Revised patch attached for review. + const char* const PGP = AP.MAI->getPrivateGlobalPrefix(); No need for the second const. Put the * to the right (const char *PGP). +Twine(Name)); space after the +. LGTM with that. Cheers, Rafael