On 8/6/2025 5:20 PM, Tobias Burnus wrote:
Hi Yuao,

thanks for the patch. — I have one nit:

Yuao Ma wrote:

This patch cleans up a duplicate test driver.
Regression-tested. OK for trunk?
...
Subject: [PATCH] fortran: cleanup duplicate tests for
  c_f_pointer_shape_driver; nfc
'; nfc'?
tests_2_driver and tests_4_driver are identical, and tests_4_driver is not used
at all. This patch clean this up, and format the driver with gcc style.
...
    * gfortran.dg/c_f_pointer_shape_tests_2_driver.c: Removed.
    * gfortran.dg/c_f_pointer_shape_tests_4_driver.c: Removed.
    * gfortran.dg/c_f_pointer_shape_tests_driver.c: New test.

This sounds as if c_f_pointer_shape_tests_driver.c is new.
But it is just the renamed _2_driver plus some formatting.
(As can be gathered from the wording above the ChangeLog).

I have bluntly updated the ChangeLog entry to

            * gfortran.dg/c_f_pointer_shape_tests_2_driver.c: Renamed to ...             * gfortran.dg/c_f_pointer_shape_tests_driver.c: ... this; format
             with gcc style.


Thank you for polishing the ChangeLog! It is much clearer now.

Yuao

Reply via email to