it'd be a bit more concise to return the string (if you prefer an out-argument, it should be a pointer to be explicit)