ask if anyone sees any drawback to this approach.

There are none except in the name -- goto -- kinda like "eval" -- scary for the scaredy

OTOH *oldname = *newname;