On 3 March 2016 at 15:56, Rui Ueyama <ruiu at google.com> wrote: > LGTM. > > As to Twine, I'd just include Twine.h or move the function definition to the > cpp file, but I think it's up to you. The calling function is a template, so it has to stay in the header. Cheers, Rafael