This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] Remove Java support
- From: Pedro Alves <palves at redhat dot com>
- To: Tom Tromey <tom at tromey dot com>, gdb-patches at sourceware dot org
- Date: Thu, 6 Oct 2016 14:32:07 +0100
- Subject: Re: [RFA] Remove Java support
- Authentication-results: sourceware.org; auth=none
- References: <1475679082-29898-1-git-send-email-tom@tromey.com>
I asked a bit internally at RH, and seems like nobody will
miss this, other than for nostalgic reasons.
On 10/05/2016 03:51 PM, Tom Tromey wrote:
> This patch removes the Java support from gdb. gcj has not seen much
> development or use for years now, and was recently removed from GCC.
> This patch changes gdb to follow; in the unlikely event that there are
> still users using gcj, they can continue to use an older gdb to debug.
> Or, they can debug in C++ mode.
>
> If this goes in, I intend to also close all the open java bugs as
> "wontfix".
Fine with me.
Do you also plan on removing the java demangling support from libiberty?
> diff --git a/gdb/testsuite/gdb.base/setshow.exp b/gdb/testsuite/gdb.base/setshow.exp
> index 2a86cdd..22d5161 100644
> --- a/gdb/testsuite/gdb.base/setshow.exp
> +++ b/gdb/testsuite/gdb.base/setshow.exp
> @@ -211,10 +211,6 @@ gdb_test "show history size" "The size of the command history is 100..*" "show h
> gdb_test "set language asm" ".*" "set language asm"
> #test show language asm
> gdb_test "show language" "The current source language is \"asm\"..*" "show language (asm)"
> -#test set language java, with a trailing space
> -gdb_test_no_output "set language java " "set language java"
> -#test show language java
> -gdb_test "show language" "The current source language is \"java\"..*" "show language (java)"
> #test completion for set language.
This was a regression test for
commit bda180e01110741f20d30b95c3ae0d93e957de8a
Author: Tom Tromey <tromey@redhat.com>
gdb
* language.c (set_language_command): Strip whitespace.
(set_type_command): Likewise.
(set_range_command): Likewise.
gdb/testsuite
* gdb.base/setshow.exp: Add tests for changes to set language, set
check range, and set check type.
Seems like we should keep it, but use some other language?
Otherwise all looked good to me.
Thanks,
Pedro Alves