One could argue that using a 'format' call at the end of all this kinda negates the advantages of using a buffer at all :)

Why not add `buff append('"')` at the start and end of the method instead?