[Bug fortran/44054] Handle -Werror, -Werror=, -fdiagnostics-show-option, !GCC$ diagnostic (pragmas) and color

["manu at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44054

--- Comment #24 from Manuel López-Ibáñez <manu at gcc dot gnu.org> ---
Author: manu
Date: Sat May 16 12:30:04 2015
New Revision: 223236

URL: https://gcc.gnu.org/viewcvs?rev=223236&root=gcc&view=rev
Log:
gcc/fortran/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054

        Replace all calls to gfc_notify_std_1 with gfc_notify_std and
        gfc_warning_1 with gfc_warning.
        * decl.c (gfc_verify_c_interop_param): Here.
        * resolve.c (resolve_branch): Here.
        (resolve_fl_derived): Here.
        * dependency.c (gfc_check_argument_var_dependency):
        * scanner.c (preprocessor_line): Use gfc_warning_now_at. Fix line
        counter and locations before and after warning.
        * gfortran.h (gfc_warning_1, gfc_warning_now_1, gfc_notify_std_1):
        Delete.
        (gfc_warning_now_at): Declare.
        * error.c (gfc_warning_1): Delete.
        (gfc_notify_std_1): Delete.
        (gfc_warning_now_1): Delete.
        (gfc_format_decoder): Handle two locations.
        (gfc_diagnostic_build_prefix): Rename as
        gfc_diagnostic_build_kind_prefix.
        (gfc_diagnostic_build_locus_prefix): Take an expanded_location
        instead of diagnostic_info.
        (gfc_diagnostic_build_locus_prefix): Add overload that takes two
        expanded_location.
        (gfc_diagnostic_starter): Handle two locations.
        (gfc_warning_now_at): New.
        (gfc_diagnostics_init): Initialize caret_chars array.
        (gfc_diagnostics_finish): Reset caret_chars array to default.

gcc/cp/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054
        * error.c (cp_diagnostic_starter): Use diagnostic_location
        function.
        (cp_print_error_function): Likewise.
        (cp_printer): Replace locus pointer with accessor function.

gcc/c/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054
        * c-objc-common.c (c_tree_printer): Replace locus pointer with
        accessor function.

gcc/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054
        * tree-pretty-print.c (percent_K_format): Replace locus pointer
        with accessor function.
        * tree-diagnostic.c (diagnostic_report_current_function): Use
        diagnostic_location function.
        (maybe_unwind_expanded_macro_loc): Likewise.
        (virt_loc_aware_diagnostic_finalizer): Likewise.
        (default_tree_printer): Replace locus pointer with accessor function.
        * diagnostic.c (diagnostic_initialize): Initialize caret_chars array.
        (diagnostic_set_info_translated): Initialize second location.
        (diagnostic_build_prefix): Use CARET_LINE_MARGIN.
        (diagnostic_show_locus): Handle two locations. Call
        diagnostic_print_caret_line.
        (diagnostic_print_caret_line): New.
        (default_diagnostic_starter): Use diagnostic_location function.
        (diagnostic_report_diagnostic): Use diagnostic_location function.
        (verbatim): Do not set text.locus.
        * diagnostic.h (struct diagnostic_info): Remove location field.
        (struct diagnostic_context): Make caret_chars an array of two.
        (diagnostic_location): New inline.
        (diagnostic_expand_location): Handle two locations.
        (diagnostic_same_line): New inline.
        (diagnostic_print_caret_line): Declare.
        (CARET_LINE_MARGIN): New constant.
        * pretty-print.c (pp_printf): Do not set text.locus.
        (pp_verbatim): Do not set text.locus.
        * pretty-print.h (MAX_LOCATIONS_PER_MESSAGE): New constant.
        (struct text_info): Replace locus pointer with locations
        array. Add accessor functions.

gcc/testsuite/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054
        * lib/gfortran-dg.exp: Update regex to handle two locations for
        the same diagnostic without caret.
        * gfortran.dg/badline.f: Test also that line numbers are correct
        before and after "left but not entered" warning.



Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/c/ChangeLog
    trunk/gcc/cp/ChangeLog
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/testsuite/ChangeLog
>From gcc-bugs-return-486433-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Sat May 16 12:31:36 2015
Return-Path: <gcc-bugs-return-486433-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 432 invoked by alias); 16 May 2015 12:31:36 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 374 invoked by uid 55); 16 May 2015 12:31:32 -0000
From: "manu at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug fortran/44054] Handle -Werror, -Werror=, -fdiagnostics-show-option, !GCC$ diagnostic (pragmas) and color
Date: Sat, 16 May 2015 12:31:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: fortran
X-Bugzilla-Version: 4.6.0
X-Bugzilla-Keywords: diagnostic
X-Bugzilla-Severity: normal
X-Bugzilla-Who: manu at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Resolution:
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields:
Message-ID: <bug-44054-4-kG6vSPveiS@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-44054-4@http.gcc.gnu.org/bugzilla/>
References: <bug-44054-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2015-05/txt/msg01273.txt.bz2
Content-length: 4776

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44054

--- Comment #25 from Manuel López-Ibáñez <manu at gcc dot gnu.org> ---
Author: manu
Date: Sat May 16 12:31:00 2015
New Revision: 223237

URL: https://gcc.gnu.org/viewcvs?rev=223237&root=gcc&view=rev
Log:
gcc/fortran/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054

        Replace all calls to gfc_notify_std_1 with gfc_notify_std and
        gfc_warning_1 with gfc_warning.
        * decl.c (gfc_verify_c_interop_param): Here.
        * resolve.c (resolve_branch): Here.
        (resolve_fl_derived): Here.
        * dependency.c (gfc_check_argument_var_dependency):
        * scanner.c (preprocessor_line): Use gfc_warning_now_at. Fix line
        counter and locations before and after warning.
        * gfortran.h (gfc_warning_1, gfc_warning_now_1, gfc_notify_std_1):
        Delete.
        (gfc_warning_now_at): Declare.
        * error.c (gfc_warning_1): Delete.
        (gfc_notify_std_1): Delete.
        (gfc_warning_now_1): Delete.
        (gfc_format_decoder): Handle two locations.
        (gfc_diagnostic_build_prefix): Rename as
        gfc_diagnostic_build_kind_prefix.
        (gfc_diagnostic_build_locus_prefix): Take an expanded_location
        instead of diagnostic_info.
        (gfc_diagnostic_build_locus_prefix): Add overload that takes two
        expanded_location.
        (gfc_diagnostic_starter): Handle two locations.
        (gfc_warning_now_at): New.
        (gfc_diagnostics_init): Initialize caret_chars array.
        (gfc_diagnostics_finish): Reset caret_chars array to default.

gcc/cp/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054
        * error.c (cp_diagnostic_starter): Use diagnostic_location
        function.
        (cp_print_error_function): Likewise.
        (cp_printer): Replace locus pointer with accessor function.

gcc/c/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054
        * c-objc-common.c (c_tree_printer): Replace locus pointer with
        accessor function.

gcc/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054
        * tree-pretty-print.c (percent_K_format): Replace locus pointer
        with accessor function.
        * tree-diagnostic.c (diagnostic_report_current_function): Use
        diagnostic_location function.
        (maybe_unwind_expanded_macro_loc): Likewise.
        (virt_loc_aware_diagnostic_finalizer): Likewise.
        (default_tree_printer): Replace locus pointer with accessor function.
        * diagnostic.c (diagnostic_initialize): Initialize caret_chars array.
        (diagnostic_set_info_translated): Initialize second location.
        (diagnostic_build_prefix): Use CARET_LINE_MARGIN.
        (diagnostic_show_locus): Handle two locations. Call
        diagnostic_print_caret_line.
        (diagnostic_print_caret_line): New.
        (default_diagnostic_starter): Use diagnostic_location function.
        (diagnostic_report_diagnostic): Use diagnostic_location function.
        (verbatim): Do not set text.locus.
        * diagnostic.h (struct diagnostic_info): Remove location field.
        (struct diagnostic_context): Make caret_chars an array of two.
        (diagnostic_location): New inline.
        (diagnostic_expand_location): Handle two locations.
        (diagnostic_same_line): New inline.
        (diagnostic_print_caret_line): Declare.
        (CARET_LINE_MARGIN): New constant.
        * pretty-print.c (pp_printf): Do not set text.locus.
        (pp_verbatim): Do not set text.locus.
        * pretty-print.h (MAX_LOCATIONS_PER_MESSAGE): New constant.
        (struct text_info): Replace locus pointer with locations
        array. Add accessor functions.

gcc/testsuite/ChangeLog:

2015-05-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR fortran/44054
        * lib/gfortran-dg.exp: Update regex to handle two locations for
        the same diagnostic without caret.
        * gfortran.dg/badline.f: Test also that line numbers are correct
        before and after "left but not entered" warning.



Modified:
    trunk/gcc/c/c-objc-common.c
    trunk/gcc/cp/error.c
    trunk/gcc/diagnostic.c
    trunk/gcc/diagnostic.h
    trunk/gcc/fortran/decl.c
    trunk/gcc/fortran/dependency.c
    trunk/gcc/fortran/error.c
    trunk/gcc/fortran/gfortran.h
    trunk/gcc/fortran/resolve.c
    trunk/gcc/fortran/scanner.c
    trunk/gcc/pretty-print.c
    trunk/gcc/pretty-print.h
    trunk/gcc/testsuite/gfortran.dg/badline.f
    trunk/gcc/testsuite/lib/gfortran-dg.exp
    trunk/gcc/tree-diagnostic.c
    trunk/gcc/tree-pretty-print.c
>From gcc-bugs-return-486434-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Sat May 16 12:33:37 2015
Return-Path: <gcc-bugs-return-486434-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 2737 invoked by alias); 16 May 2015 12:33:37 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 2703 invoked by uid 55); 16 May 2015 12:33:33 -0000
From: "tkoenig at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug fortran/66113] Variable n cannot appear in the expression with nested blocks
Date: Sat, 16 May 2015 12:33:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: fortran
X-Bugzilla-Version: 6.0
X-Bugzilla-Keywords: rejects-valid
X-Bugzilla-Severity: normal
X-Bugzilla-Who: tkoenig at gcc dot gnu.org
X-Bugzilla-Status: UNCONFIRMED
X-Bugzilla-Resolution:
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields:
Message-ID: <bug-66113-4-f5dRAJDNFW@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-66113-4@http.gcc.gnu.org/bugzilla/>
References: <bug-66113-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2015-05/txt/msg01274.txt.bz2
Content-length: 713

https://gcc.gnu.org/bugzilla/show_bug.cgi?idf113

--- Comment #5 from Thomas Koenig <tkoenig at gcc dot gnu.org> ---
Author: tkoenig
Date: Sat May 16 12:33:01 2015
New Revision: 223238

URL: https://gcc.gnu.org/viewcvs?rev"3238&root=gcc&view=rev
Log:
2015-05-16  Thomas Koenig  <tkoenig@gcc.gnu.org>

        PR fortran/66113
        * expr.c (is_parent_of_current_ns):  New function.
        (check_restricted):  Use it.

2015-05-16  Thomas Koenig  <tkoenig@gcc.gnu.org>

        PR fortran/66113
        * gfortran.dg/block_14.f90:  New test.


Added:
    trunk/gcc/testsuite/gfortran.dg/block_14.f90
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/expr.c
    trunk/gcc/testsuite/ChangeLog