するなら
void swap(char **x,char **y)
としない？