This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, fortran] New take on PR 82373
- From: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
- To: Thomas Koenig <tkoenig at netcologne dot de>
- Cc: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 12 Oct 2017 12:26:36 -0700
- Subject: Re: [patch, fortran] New take on PR 82373
- Authentication-results: sourceware.org; auth=none
- References: <e3e1da07-0dce-d198-3694-7040bd2febe4@netcologne.de>
- Reply-to: sgk at troutmask dot apl dot washington dot edu
On Thu, Oct 12, 2017 at 08:21:46PM +0200, Thomas Koenig wrote:
>
> after some thought, I think the PR can be fixed by something
> far less invasive than my previous patch.
>
> The new version of the patch simply issues an error for a
> non-printable character (which should never be legal).
> Anything else should be caught by other error reporting
> routines.
>
> OK for trunk?
>
I like this patch much better than the last. It's
ok with me, but you might want give 24 hours to let
the world rotate before committing (ie., allow others
to comment).
--
Steve
20170425 https://www.youtube.com/watch?v=VWUpyCsUKR4
20161221 https://www.youtube.com/watch?v=IbCHE-hONow