Re: RF[CT]: IMA repairs 2/n: maybe_apply_renaming_pragma rawrite

[Andrew Pinski <apinski at apple dot com>]

On Jun 14, 2004, at 4:53 PM, Andrew Pinski wrote:

> I decided to fix up this patch for Zack and I found it was just a
> simple off by one error when handling maybe_apply_renaming_pragma.
>
> Bootstrapped on powerpc-apple-darwin with no regressions plus I tried
> the simple testcase which Rainer Orth gave and it looked to be fixed.
>
> OK to commit if someone tests on Solaris and Tru64 (OSF) and there are
> no regressions because of this patch?


I had forgot to say the only difference between this patch and the one
which Zack sent out is:

+      /* If we have a null character as the last charactor in the
+	 array, then the length of the string is smaller by one.  */
+      if (prefix[plen - 1] == '\0')
+	plen--;
+
+      if (id[ilen - 1] == '\0')
+	ilen--;


Thanks,
Andrew Pinski