>>>>> "Ian" == Ian Rogers <[EMAIL PROTECTED]> writes:
Ian> Please let me know if you think this patch is suitable for
Ian> inclusion.
It looks fine. I do have one nit, which is that we put spaces around
operators... this problem is pervasive in the patch, but here's one
example:
Ian> + newStr[x-offset] = newChar;
That should read
newStr[x - offset] = newChar;
Tom
