This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH diagnostics/Fortran] Implement Fortran prefix/caret style using the common diagnostics machinery
- From: Dodji Seketeli <dodji at redhat dot com>
- To: Manuel LÃpez-IbÃÃez <lopezibanez at gmail dot com>
- Cc: Gcc Patch List <gcc-patches at gcc dot gnu dot org>, Tobias Burnus <burnus at net-b dot de>, "fortran\ at gcc dot gnu dot org List" <fortran at gcc dot gnu dot org>
- Date: Wed, 20 Aug 2014 10:06:22 +0200
- Subject: Re: [PATCH diagnostics/Fortran] Implement Fortran prefix/caret style using the common diagnostics machinery
- Authentication-results: sourceware.org; auth=none
- References: <CAESRpQDk1v4BwQX5jzxFGQEADFb2rsEMsZxr_yc6UU0KDeFqGA at mail dot gmail dot com>
Hello Manuel,
Manuel LÃpez-IbÃÃez <lopezibanez@gmail.com> writes:
> Hi,
>
> This patch is relative to this one here:
> https://gcc.gnu.org/ml/gcc-patches/2014-08/msg01652.html
>
> It implements the Fortran style of prefix and caret line in the
> gfc_diagnostic_starter by using the common pretty-printer.
>
> This requires making configurable the caret character in the
> diagnostics machinery.
[...]
> 2014-08-19 Manuel LÃpez-IbÃÃez <manu@gcc.gnu.org>
>
> PR fortran/44054
> * diagnostic.c: Set default caret.
> (diagnostic_show_locus): Use it. Tell pretty-printer that a new
> line is needed.
> * diagnostic.h (struct diagnostic_context):
> Bootstrapped and regression tested on x86_64-linux-gnu.
>
> OK?
The generic diagnostics part is OK.
Thanks!
Cheers.
--
Dodji