This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Re: [PATCH] Fix unused but set warnings in Java FE
- From: Andrew Haley <aph at redhat dot com>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: tromey at redhat dot com, gcc-patches at gcc dot gnu dot org, java-patches at gcc dot gnu dot org
- Date: Sat, 28 Nov 2009 14:55:16 +0000
- Subject: Re: [PATCH] Fix unused but set warnings in Java FE
- References: <20091128113107.GE22813@hs20-bc2-1.build.redhat.com>
Jakub Jelinek wrote:
> Hi!
>
> This patch fixes some -Wunused-but-set-* warnings in Java FE.
> gcc/java/builtins.c:325:7: warning: variable âvalue_typeâ set but not used
> gcc/java/builtins.c:349:7: warning: variable âvalue_typeâ set but not used
> gcc/java/builtins.c:412:3: warning: variable âthis_argâ set but not used
> gcc/java/javaop.def:278:1: warning: variable âsaw_indexâ set but not used
> gcc/java/javaop.def:311:1: warning: variable âsaw_indexâ set but not used
> warnings are still left, I'd appreciate help with either fixing those or
> finding workarounds.
>
> Bootstrapped/regtested on x86_64-linux, ok for trunk?
>
> 2009-11-28 Jakub Jelinek <jakub@redhat.com>
>
> * jvspec.c (lang_specific_driver): Remove unused
> saw_verbose_flag variable.
> * jcf-dump.c (main): Remove unused general_purpose_bits
> variable.
> * builtins.c (initialize_builtins): Remove unused float_ftype_float
> variable.
> * expr.c (java_stack_pop): Remove unused val variable.
> (build_jni_stub): Remove unused res_type variable.
> * verify-impl.c (check_field_constant): Remove unused len variable.
Sure, this stuff is all pretty much obvious/trivial.
Andrew.