> Looks OK to me except for:
>
>> - for (; i > 0; i--)
>> + for (; i > 0;)
>
> Might as well just make that a while loop.
Indeed! Committed with a while loop, thanks for the review!
FX
On Sat, Mar 3, 2012 at 4:08 PM, FX wrote:
> The attached patch fixes PR 36160
> (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36160). It should correctly
> account for wide characters when display error loci. I'm not sure if we can
> check that in the testsuite harness, but you can manually see
The attached patch fixes PR 36160
(http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36160). It should correctly
account for wide characters when display error loci. I'm not sure if we can
check that in the testsuite harness, but you can manually see it at work on the
attached test.f90.
Bootstrapped