This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, Fortran] PR 79968: merge similar diagnostics containing -fdec-structure
- From: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- To: Fritz Reese <fritzoreese at gmail dot com>
- Cc: fortran <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 17 May 2017 19:44:15 +0100
- Subject: Re: [PATCH, Fortran] PR 79968: merge similar diagnostics containing -fdec-structure
- Authentication-results: sourceware.org; auth=none
- References: <CAE4aFAm511zc+o=FK1rd+4xjy1i03tZMe0ja-hKhZh3PoJGG+g@mail.gmail.com>
Hi Fritz,
Yes, that's good for trunk.
Thanks
Paul
On 17 May 2017 at 16:51, Fritz Reese <fritzoreese@gmail.com> wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79968
>
> All,
>
> The PR suggests unifying diagnostics of the form "%s at %C is a DEC
> extension, enable with ..." I think the patch is obvious/trivial and
> speaks for itself. I plan to commit the attached to trunk shortly,
> barring any complaints.
>
> (Nb. even with the change in diagnostics, the applicable testcases
> (dec_io_3.f90 and dec_static_3.f90) are not regressed as they match
> only the "is a DEC extension" portion of the error message.)
>
> ---
> Fritz Reese
>
> 2017-05-17 Fritz Reese <fritzoreese@gmail.com>
>
> PR fortran/79968
>
> gcc/fortran/ChangeLog:
>
> PR fortran/79968
> * decl.c (match_attr_spec, gfc_match_automatic,
> gfc_match_static, gfc_match_structure_decl): Unify diagnostic
> errors regarding -fdec options.
> * io.c (match_dec_etag, match_dec_vtag, match_dec_ftag): Ditto.
--
"If you can't explain it simply, you don't understand it well enough"
- Albert Einstein