* [PATCH] gdb/mi: remove warning about mi1
@ 2023-09-12 17:37 Simon Marchi
2023-09-12 18:03 ` Tom Tromey
0 siblings, 1 reply; 3+ messages in thread
From: Simon Marchi @ 2023-09-12 17:37 UTC (permalink / raw)
To: gdb-patches; +Cc: Simon Marchi
Remove a warning about mi1. mi1 was removed in 975249ff4e26 ("Remove MI
version 1"). It is no longer possible to reach this warning, since
trying to use interpreter mi1 bails out before:
$ ./gdb -nx -q --data-directory=data-directory -i mi1
Interpreter `mi1' unrecognized
Change-Id: Ie43b21e01bca1407995150c729531a70ee662003
---
gdb/interps.c | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/gdb/interps.c b/gdb/interps.c
index adac98125239..f91b2b62c1ba 100644
--- a/gdb/interps.c
+++ b/gdb/interps.c
@@ -132,16 +132,11 @@ interp_set (struct interp *interp, bool top_level)
if (interpreter_p != interp->name ())
interpreter_p = interp->name ();
- bool warn_about_mi1 = false;
-
/* Run the init proc. */
if (!interp->inited)
{
interp->init (top_level);
interp->inited = true;
-
- if (streq (interp->name (), "mi1"))
- warn_about_mi1 = true;
}
/* Do this only after the interpreter is initialized. */
@@ -151,11 +146,6 @@ interp_set (struct interp *interp, bool top_level)
clear_interpreter_hooks ();
interp->resume ();
-
- if (warn_about_mi1)
- warning (_("MI version 1 is deprecated in GDB 13 and "
- "will be removed in GDB 14. Please upgrade "
- "to a newer version of MI."));
}
/* Look up the interpreter for NAME. If no such interpreter exists,
base-commit: f8ab027008863bd06513053ea9f84e1116c1cf73
--
2.42.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] gdb/mi: remove warning about mi1
2023-09-12 17:37 [PATCH] gdb/mi: remove warning about mi1 Simon Marchi
@ 2023-09-12 18:03 ` Tom Tromey
2023-09-12 18:14 ` Simon Marchi
0 siblings, 1 reply; 3+ messages in thread
From: Tom Tromey @ 2023-09-12 18:03 UTC (permalink / raw)
To: Simon Marchi via Gdb-patches; +Cc: Simon Marchi
>>>>> "Simon" == Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:
Simon> Remove a warning about mi1. mi1 was removed in 975249ff4e26 ("Remove MI
Simon> version 1"). It is no longer possible to reach this warning, since
Simon> trying to use interpreter mi1 bails out before:
Simon> $ ./gdb -nx -q --data-directory=data-directory -i mi1
Simon> Interpreter `mi1' unrecognized
Thank you.
Approved-By: Tom Tromey <tom@tromey.com>
Tom
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] gdb/mi: remove warning about mi1
2023-09-12 18:03 ` Tom Tromey
@ 2023-09-12 18:14 ` Simon Marchi
0 siblings, 0 replies; 3+ messages in thread
From: Simon Marchi @ 2023-09-12 18:14 UTC (permalink / raw)
To: Tom Tromey, Simon Marchi via Gdb-patches
On 9/12/23 14:03, Tom Tromey wrote:
>>>>>> "Simon" == Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:
>
> Simon> Remove a warning about mi1. mi1 was removed in 975249ff4e26 ("Remove MI
> Simon> version 1"). It is no longer possible to reach this warning, since
> Simon> trying to use interpreter mi1 bails out before:
>
> Simon> $ ./gdb -nx -q --data-directory=data-directory -i mi1
> Simon> Interpreter `mi1' unrecognized
>
> Thank you.
> Approved-By: Tom Tromey <tom@tromey.com>
>
> Tom
Thanks, pushed.
Simon
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-09-12 18:14 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-12 17:37 [PATCH] gdb/mi: remove warning about mi1 Simon Marchi
2023-09-12 18:03 ` Tom Tromey
2023-09-12 18:14 ` Simon Marchi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).