[Fwd: [PATCH, libgfortran] Fix for PR24699, PR24700, PR24719]]

Jerry DeLisle jvdelisle@verizon.net
Fri Nov 11 18:48:00 GMT 2005


Paul Thomas wrote:
> :REVIEWMAIL:
> 
> Jerry,
> 
> I had to add your patch by hand because all 5 chunks failed.  I believe 
> that this is because of whitespace differences; your version has no tabs 
> and so the context was not recognized.  Please correct this.

Curious, there are tabs in my source and that is how I have set my editor. 
Maybe its my diff command.  I will check it.

> 
> Apart from this, the patch is fine.  It even does what it should do!  
> All three PRs are fixed.  I don't recall - did NIST check out OK with 
> the patch?  If you did not do it, please do not commit until every last 
> NIST test runs correctly.
> 
> pr24699.f90 is OK - it checks out the problem with aermod correctly.
> pr24719.f90 is OK, except for the print statement on line 12. Please 
> remove it.

I noticed that too, it is removed.

> 
> Why don't you take the original offering to Bugzilla as your testcase 
> for pr24785?  This works:
> 
> ! { dg-do run }
> ! ( dg-output "^" }
> !
> ! Test fix for pr24785 - EOR used to scrub the 2X.
> ! Reduced from PR example submitted by Harald Anlauf <anlauf@gmx.de>
> !
>      program x_with_advance_bug
>      write (*,'(A,2X)',  advance="no") "<"
>      write (*,'(A)') ">" ! { dg-output "<  >" }
>      end
>

This is great.  Now I have learned another "dg-" feature I can use.

Thanks,

Jerry



More information about the Fortran mailing list