This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH, Fortran] pad char to int conversions with spaces instead of zeros (legacy)
- From: Jerry DeLisle <jvdelisle at charter dot net>
- To: Fritz Reese <fritzoreese at gmail dot com>, Jakub Jelinek <jakub at redhat dot com>
- Cc: mark dot eggleston at codethink dot co dot uk, fortran <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Jeff Law <law at redhat dot com>
- Date: Wed, 5 Dec 2018 18:27:00 -0800
- Subject: Re: [PATCH, Fortran] pad char to int conversions with spaces instead of zeros (legacy)
- References: <firstname.lastname@example.org> <20181204151130.GB12380@tucnak> <CAE4aFAnU22uAP8W5K+FdQUJEarox_wqBbD0TGo_RynawoHeKEA@mail.gmail.com>
On 12/4/18 9:04 AM, Fritz Reese wrote:
On Tue, Dec 4, 2018 at 10:12 AM Jakub Jelinek <email@example.com> wrote:
Just a couple of random comments.
-fdec-pad-with-spaces option name doesn't look right, because it doesn't say
what the option affects. So perhaps have transfer in the option name?
Wouldn't it be better to allow specifying whatever character you want to pad
with, so that users can choose something even different?
I concur with this. In that case some option like -ftransfer-pad-char=
may be a more appriopriate name, where -fdec may enable a default
transfer-pad-char of \x20 rather than \x00.
I disagree completely. I assume the idea of -fdec-pad-with-spaces is to
accomodate some old dec fortran code. The only reason to use some other
character is if someone is writing new dec fortran code, which is
implying encouraging people to be writing non standard conforming code.
Even if it is conforming in the sense that it is processor dependent you
are encouraging people to create new non portable code across compilers.
Please just stay consistent with Intel.
This whole padding business just stinks to me. There are better ways to
accomplish these results without using transfer to convert ascii strings
into bit patterns or whatever the heck some programmer is trying to do
here, like maybe use 'ABCE ' if thats what is really needed. And,
please everyone please quit fiddling with the compiler to fix problems
in the source code. Are people so lazy or such cheapskates they won't do
this the right way and update the damn source code if it needs to be used.
We have truly more serious and real problems/bugs in gfortran that
people should be spending the scarce resources on and not this junk.