Fix ARI violations in c++compile
authorKeith Seitz <keiths@redhat.com>
Thu, 30 Aug 2018 16:26:47 +0000 (09:26 -0700)
committerKeith Seitz <keiths@redhat.com>
Sat, 1 Sep 2018 19:04:29 +0000 (12:04 -0700)
This patch fixes two violations of the ARI (use of ATTRIBUTE_UNUSED and
"%ll").

gdb/ChangeLog

* compile/compile-cplus-types.c (compile_cplus_debug_output_1): Use
pulongest instead of "%lld".
* compile/compile-cplus-symbols.c (gcc_cplus_convert_symbol): Remove
ATTRIBUTE_UNUSED.

gdb/ChangeLog
gdb/compile/compile-cplus-symbols.c
gdb/compile/compile-cplus-types.c

index 83348d52a941574cd1267d7001750ae68f9d508b..128c0e859736f7c5edcdeb905e369d74cfe90bb5 100644 (file)
@@ -1,3 +1,10 @@
+2018-09-01  Keith Seitz  <keiths@redhat.com>
+
+       * compile/compile-cplus-types.c (compile_cplus_debug_output_1): Use
+       pulongest instead of "%lld".
+       * compile/compile-cplus-symbols.c (gcc_cplus_convert_symbol): Remove
+       ATTRIBUTE_UNUSED.
+
 2018-08-31  Tom Tromey  <tom@tromey.com>
 
        * dwarf2read.c (dwarf2_add_field): Set the TYPE_LENGTH of the
index 0f849fe36558ea871939bcb0cb9b2730722b7fb1..234b97783bc055ed6271f7a95a5db34a83b123df 100644 (file)
@@ -335,7 +335,7 @@ convert_symbol_bmsym (compile_cplus_instance *instance,
 void
 gcc_cplus_convert_symbol (void *datum,
                          struct gcc_cp_context *gcc_context,
-                         enum gcc_cp_oracle_request request ATTRIBUTE_UNUSED,
+                         enum gcc_cp_oracle_request request,
                          const char *identifier)
 {
   if (compile_debug)
index 7fc413606e27dba54281543379b15a1e76e03c9a..271cc665f85c5739946415a68ef73a9d584a12a6 100644 (file)
@@ -1244,9 +1244,9 @@ compile_cplus_instance::gcc_cplus_leave_scope
    gcc_decl.  */
 
 static void
-compile_cplus_debug_output_1 (gcc_type arg)
+compile_cplus_debug_output_1 (ULONGEST arg)
 {
-  fprintf_unfiltered (gdb_stdlog, "%lld", arg);
+  fprintf_unfiltered (gdb_stdlog, "%s", pulongest (arg));
 }
 
 static void