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