On 5/28/25 10:09 AM, Steve Kargl wrote:
On Wed, May 28, 2025 at 08:11:05AM -0700, Jerry D wrote:
The attached patch is simple and self explanatory in the git log entry.

Regression tested on X86_64-linux-gnu.

OK for trunk?


Yes, with one question.

commit 845768cbead03f76265e491bcf5ea6de7020ff39
Author: Jerry DeLisle <jvdeli...@gcc.gnu.org>
Date:   Wed May 28 07:56:12 2025 -0700

     Fortran: Adjust handling of optional comma in FORMAT.
This change adjusts the error messages for optional commas
             in format strings to give a warning at compile time unless
             -std=legacy is used. This is more consistant with the
             runtime library. The comma seprator should really not be
             encouraged as it is non-standard fortran.

Is that last sentence correct?  I would think that the
comma separator is not only encouraged, but it is required


Pushed with fixing the comment. And ... with my fat fingers I reversed two digits of the PR number in the log and I do not know how to fix that.

Jerry

Reply via email to