Fix complaint string formatting on stap-probe.c
authorSergio Durigan Junior <sergiodj@redhat.com>
Thu, 16 May 2019 20:20:39 +0000 (16:20 -0400)
committerSergio Durigan Junior <sergiodj@redhat.com>
Thu, 16 May 2019 20:26:29 +0000 (16:26 -0400)
I think the string formatting for complaints was messed up by Tom's
patch to simplify the complaint mechanism.  This small and trivial
patch fixes them.

Pushed as obvious.

gdb/ChangeLog:
2019-05-16  Sergio Durigan Junior  <sergiodj@redhat.com>

* stap-probe.c (handle_stap_probe): Fix complaint formatting.
(stap_static_probe_ops::get_probes): Likewise.

gdb/ChangeLog
gdb/stap-probe.c

index c89b703..faee999 100644 (file)
@@ -1,5 +1,10 @@
 2019-05-16  Sergio Durigan Junior  <sergiodj@redhat.com>
 
+       * stap-probe.c (handle_stap_probe): Fix complaint formatting.
+       (stap_static_probe_ops::get_probes): Likewise.
+
+2019-05-16  Sergio Durigan Junior  <sergiodj@redhat.com>
+
        * stap-probe.c (stap_parse_register_operand): Make "if (*p->arg ==
        '-')" and "else if".
        (stap_parse_single_operand): Join checks for
index db92315..5261a1a 100644 (file)
@@ -1512,12 +1512,11 @@ handle_stap_probe (struct objfile *objfile, struct sdt_note *el,
   /* Making sure there is a name.  */
   if (name == NULL)
     {
-      complaint (_("corrupt probe name when "
-                                       "reading `%s'"),
+      complaint (_("corrupt probe name when reading `%s'"),
                 objfile_name (objfile));
 
       /* There is no way to use a probe without a name or a provider, so
-        returning zero here makes sense.  */
+        returning here makes sense.  */
       return;
     }
   else
@@ -1549,11 +1548,10 @@ handle_stap_probe (struct objfile *objfile, struct sdt_note *el,
       || (memchr (probe_args, '\0', (char *) el->data + el->size - name)
          != el->data + el->size - 1))
     {
-      complaint (_("corrupt probe argument when "
-                                       "reading `%s'"),
+      complaint (_("corrupt probe argument when reading `%s'"),
                 objfile_name (objfile));
       /* If the argument string is NULL, it means some problem happened with
-        it.  So we return 0.  */
+        it.  So we return.  */
       return;
     }
 
@@ -1661,8 +1659,7 @@ stap_static_probe_ops::get_probes
     {
       /* If we are here, it means we have failed to parse every known
         probe.  */
-      complaint (_("could not parse SystemTap probe(s) "
-                                       "from inferior"));
+      complaint (_("could not parse SystemTap probe(s) from inferior"));
       return;
     }
 }