Well I think I'm weary of this because it really only gets used once and seems to be for the XML -> Hash case. Id say either inline it or rename it to something more specific.