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.

and pushed it as r16-3031-gde2eaf63ad4334 / 
https://gcc.gnu.org/r16-3031-gde2eaf63ad4334

Thanks for the cleanup!

Tobias

Reply via email to