Am 06.08.2014 18:50, schrieb Manuel López-Ibáñez:
This is the first step for moving Fortran to use the common
diagnostics machinery. This patch makes Fortran use the common
machinery for those warnings that don't have a location or a
controlling option.
Bootstrapped and regression tested on x86_64-linux-gnu.
OK?

Sorry for replying belated, I was in vacation and seemingly, a few others as well.

The patch looks good to me, except for gfc-diagnostic.def which contains the file content twice. With that fixed, the patch is OK. Thanks a lot!

Tobias

2014-08-03  Manuel López-Ibáñez  <m...@gcc.gnu.org>

     PR fortran/44054
c-family/
     * c-format.c: Handle Fortran flags.
     * diagnostic.c (build_message_string): Make it extern.
     * diagnostic.h (build_message_string): Make it extern.
fortran/
     * gfortran.h: Define GCC_DIAG_STYLE.
     (gfc_diagnostics_init,gfc_warning_cmdline): Declare.
     * trans-array.c: Include gfortran.h before diagnostic-core.h.
     * trans-expr.c: Likewise.
     * trans-openmp.c: Likewise.
     * trans-const.c: Likewise.
     * trans.c: Likewise.
     * trans-types.c: Likewise.
     * f95-lang.c: Likewise.
     * trans-decl.c: Likewise.
     * trans-io.c: Likewise.
     * trans-intrinsic.c: Likewise.
     * error.c: Include diagnostic.h and diagnostic-color.h.
     (gfc_diagnostic_build_prefix): New.
     (gfc_diagnostic_starter): New.
     (gfc_diagnostic_finalizer): New.
     (gfc_warning_cmdline): New.
     (gfc_diagnostics_init): New.
     * gfc-diagnostic.def: New.
     * options.c (gfc_init_options): Call gfc_diagnostics_init.
     (gfc_post_options): Use gfc_warning_cmdline.

Reply via email to