This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch,libgfortran] Fix NIST fm906.for - redux


Paul Thomas wrote:
> Correct version that I propose to commit.

This adresses exactly what I wanted to ask in my just-finished response :-)

>>! Test a new record after the.comma (the original bug).
                               ^ typo
>>
>> rewind (10)
>> write (10, *) " ( 99.0   ,"
>> write (10, *) "   999.0  )"
>> rewind (10)
>> read (10,*) a
>> if (a.ne.(99.0,999.0)) call abort ()
>>
>>! Test a new record before the.comma 
                                ^ same
The line has a trailing space, but we don't require any coding style in
testcases, I jsut happened to click on it ;)

>>! Test a new records before and after the.comma

I'm beginning to wonder if there's a grammar rule I don't know about ...

I think the plural would be more adequate, and please supplement a test which
tests multiple lines before and after.

Otherwise ok, thanks.

- Tobi


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]