Minor detail, but is there a reason not to use StringBuilder here?