commit 41daaea2ccaef06fdb648e7f2a3095d8feac2aa5
author Tom Tromey <tromey@adacore.com> Tue Dec 13 08:58:50 2022 -0700
committer Tom Tromey <tromey@adacore.com> Tue Dec 13 08:58:50 2022 -0700
tree 119c6ac27c76b4d745b7404c3a169f4fa1ba281f
parent dc3fb44540b586c02ec2f841e106a8d2cc2a422d

Remove two unnecessary casts

A couple of calls to parse_probe_linespec had an unnecessary cast.  I
suspect this cast was never needed, but once commands were changed to
take a 'const' argument, they became completely obsolete.  Tested by
rebuilding.

gdb/probe.c

diff --git a/gdb/probe.c b/gdb/probe.c
index ec88452..4193f9f 100644
--- a/gdb/probe.c
+++ b/gdb/probe.c
@@ -617,7 +617,7 @@ enable_probes_command (const char *arg, int from_tty)
 {
   std::string provider, probe_name, objname;
 
-  parse_probe_linespec ((const char *) arg, &provider, &probe_name, &objname);
+  parse_probe_linespec (arg, &provider, &probe_name, &objname);
 
   std::vector<bound_probe> probes
     = collect_probes (objname, provider, probe_name, &any_static_probe_ops);
@@ -652,7 +652,7 @@ disable_probes_command (const char *arg, int from_tty)
 {
   std::string provider, probe_name, objname;
 
-  parse_probe_linespec ((const char *) arg, &provider, &probe_name, &objname);
+  parse_probe_linespec (arg, &provider, &probe_name, &objname);
 
   std::vector<bound_probe> probes
     = collect_probes (objname, provider, probe_name, &any_static_probe_ops);