Don't use uninitialized variables. Don't do crazy trickery such as s[i] = temp[i++];