This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] Remove Java support


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]