Hi Jerry!

Am 14.12.25 um 23:56 schrieb Jerry D:
Hi all,

See attached patch and change log below.

Took we a while to sort this one out. Ultimately, my fix for PR105361 was wrong and we were calling hit_eof in the wrong place for the wrong reasons. This is also a regression because the new testcase passes on 14 and fails on 15.

Regression tested on x86_64_linux_gnu.

OK for mainline? Then backport to 15 after some burn in.

Regards,

Jerry

commit f3df5d8f6b775f24b6090840a1b604d1582f875a
Author: Jerry DeLisle <[email protected]>
Date:   Sun Dec 14 13:23:36 2025 -0800

Fortran: Fix bad read involving extra input text.

     The problem here involved DTIO mixed with non-DTIO
     variables in list formatted reads.  The previous fix to
     PR105361 broke the test case here by mis-handling the
     end of file conditions. It was found that the code could
     be significantly reduced as well.

     PR libfortran/122936

     libgfortran/ChangeLog:

        * io/list_read.c (finish_list_read): Remove the use of hit_eof
        and free_line. Simplify the logic. Add comments to clarify.

     gcc/testsuite/ChangeLog:

        * gfortran.dg/pr122936.f90: New test.


Yes, this is OK for mainline and backport.

Thanks for the patch!

Harald

Reply via email to