This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: Ping : [Patch, fortran] PR48298 - [F03] User-Defined Derived-Type IO (DTIO)
- From: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- To: Andre Vehreschild <vehre at gmx dot de>
- Cc: GCC-Fortran-ML <fortran at gcc dot gnu dot org>, jerry DeLisle <jvdelisle at charter dot net>
- Date: Tue, 30 Aug 2016 23:29:44 +0200
- Subject: Re: Ping : [Patch, fortran] PR48298 - [F03] User-Defined Derived-Type IO (DTIO)
- Authentication-results: sourceware.org; auth=none
- References: <CAGkQGiKvbLD8azLWWaxMSSpSXWb_qHWFFA3=MwG8VFK+-mZ+wg@mail.gmail.com> <CAGkQGiKei+bpy69xwL0idyGW4ZvPeod-u6a7RT=s3NV4_wEx3w@mail.gmail.com> <CAGkQGiLvuhqvmEgF9ZXNz6E_qPfOYsC_Yd7oO4h4=T8g=CUvjQ@mail.gmail.com> <CAGkQGi+gK_6Z0yQeCM92b_mtB+pinsBiCVv1teBSbtUeortrNQ@mail.gmail.com> <20160830183933.1d2c3202@vepi2> <CAGkQGiJ0Y+DNP38-Df_hf_fwK1VP+hxaGasHMHsrPiWhz931pQ@mail.gmail.com> <F1B8D9B0-3044-4C22-B76A-B3F670987945@gmx.de>
Hi Andre,
Ah yes! Both Jerry and I use an editor that does the whitespace thing
automatically. Sorry, I misunderstood what you were saying. I have, in
the past, used -wcp but this results in difficulties with application
with patch, unless your know to use the -l option.
Cheers
Paul
On 30 August 2016 at 22:12, Andre Vehreschild <vehre@gmx.de> wrote:
> Hi Paul,
>
>
>>> man, you really could have done a patch just correcting style. That
>>> would have reduced the patch size by half. :-)
>>
>>Sorry, what do you mean? What's up with the style?
>
> Nothing, now that you've fixed it. I meant that half of the locations marked changed in the patch remove trailing whitespace. When you had split those changes from the patch about the dtio, it would have been half as large and I think more people would have taken a look. A large patch scares them away. Just a thought.
>
> - Andre
> --
> Andre Vehreschild * vehre ad gmx dot de
--
The difference between genius and stupidity is; genius has its limits.
Albert Einstein