don't pass in string