This still isn't quite right.  The `memmove` isn't moving the `\0` so it ends up stuck in a loop repeatedly inserting the last character.  I assume `newtext . clone()` is returning a null terminated string so it should be `memmove(sptr, sptr + 1, strlen(sptr));` as `strlen()` doesn't include the terminating null character.