Really nitpicky, but why assign this to `Xml`, instead of just returning the string?