re PR fortran/44054 (Handle -Werror, -Werror=, -fdiagnostics-show-option, !GCC$ diagn...
authorManuel López-Ibáñez <manu@gcc.gnu.org>
Sat, 16 May 2015 12:30:04 +0000 (12:30 +0000)
committerManuel López-Ibáñez <manu@gcc.gnu.org>
Sat, 16 May 2015 12:30:04 +0000 (12:30 +0000)
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.

From-SVN: r223236

gcc/ChangeLog
gcc/c/ChangeLog
gcc/cp/ChangeLog
gcc/fortran/ChangeLog
gcc/testsuite/ChangeLog

index 84e282a0e8486d3257064dd9c6224e2bad2820dc..1f2a4ab686bae34868ab46c7e4f2dd1217d10645 100644 (file)
@@ -1,3 +1,35 @@
+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.
+
 2015-05-16  Kugan Vivekanandarajah  <kuganv@linaro.org>
            Zhenqiang Chen  <zhenqiang.chen@linaro.org>
 
index ebf9759bc9b699b1b2d24c95fabf60bd6d48bd9d..6ef290f79333f7a0970d552aec50a505dab09812 100644 (file)
@@ -1,3 +1,9 @@
+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.
+
 2015-05-14  Marek Polacek  <polacek@redhat.com>
 
        PR c/66066
index b119f7bc4f2a912c50456e4f8754b2c05232ce36..210b177046327488b55d7c9b15f207f528de29ec 100644 (file)
@@ -1,3 +1,11 @@
+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.
+
 2015-05-12  David Malcolm  <dmalcolm@redhat.com>
 
        * parser.c (cp_parser_selection_statement): Add location and
index c06627e32c917ad4d1490884394cf162dcacd93b..7cff4760cac3426280f967818f37e105bbe6188d 100644 (file)
@@ -1,3 +1,33 @@
+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.
+
 2015-05-16  Mikael Morin  <mikael@gcc.gnu.org
            Paul Thomas  <pault@gcc.gnu.org>
 
index 430e6bd72878cd918bb4a1e1f51eca65406efc5d..7260828346d5d7ec2ccafd7b963376e69cff94f2 100644 (file)
@@ -1,3 +1,11 @@
+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.
+
 2015-05-16  Kugan Vivekanandarajah  <kuganv@linaro.org>
            Zhenqiang Chen  <zhenqiang.chen@linaro.org>