Trouble with typecasting

Does this make any sense?

short byteswap(short x){

char y[3];
(short)y[1] = x;
y[0] = y[2];
return (short)y[0];

}