This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Re: Eliminate natSocketChannelImpl.cc warnings
- From: Michael Koch <konqueror at gmx dot de>
- To: Matt Kraai <kraai at alumni dot cmu dot edu>
- Cc: java-patches at gcc dot gnu dot org
- Date: Wed, 18 Jun 2003 08:29:40 +0200
- Subject: Re: Eliminate natSocketChannelImpl.cc warnings
- References: <20030617215436.GD15127@ftbfs.org>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Am Dienstag, 17. Juni 2003 23:54 schrieb Matt Kraai:
> Howdy,
>
> natSocketChannelImpl.cc generates the following warnings
>
> ../../../../gcc/libjava/gnu/java/nio/natSocketChannelImpl.cc: In
> static member function `static jint
> gnu::java::nio::SocketChannelImpl::SocketConnect(__java_int,
> java::net::InetAddress*, __java_int)':
> ../../../../gcc/libjava/gnu/java/nio/natSocketChannelImpl.cc:96:
> warning: unused parameter `jint fd'
> ../../../../gcc/libjava/gnu/java/nio/natSocketChannelImpl.cc:96:
> warning: unused parameter `java::net::InetAddress*addr'
> ../../../../gcc/libjava/gnu/java/nio/natSocketChannelImpl.cc:96:
> warning: unused parameter `jint port'
> ../../../../gcc/libjava/gnu/java/nio/natSocketChannelImpl.cc: In
> static member function `static jint
> gnu::java::nio::SocketChannelImpl::SocketBind(__java_int,
> java::net::InetAddress*, __java_int)':
> ../../../../gcc/libjava/gnu/java/nio/natSocketChannelImpl.cc:104:
> warning: unused parameter `jint fd'
> ../../../../gcc/libjava/gnu/java/nio/natSocketChannelImpl.cc:104:
> warning: unused parameter `java::net::InetAddress*addr'
> ../../../../gcc/libjava/gnu/java/nio/natSocketChannelImpl.cc:104:
> warning: unused parameter `jint port'
>
> Both functions are stubs that ignore their arguments, so the
> following patch marks the parameters with
>
> __attribute__ ((__unused__))
>
> Bootstrapped and regression tested on powerpc-unknown-linux-gnu,
> and I checked that the warnings were eliminated.
>
> I can commit to the GCC repository but not to the Classpath
> repository.
>
> OK to commit?
My next patch to the NIO stuff will remove this file totally so no
need to fix this yet.
Michael
- --
Homepage: http://www.worldforge.org/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)
iD8DBQE+8AbXWSOgCCdjSDsRAvCeAJ9crLY6kvQ1r12gm2wnGz1N99I34QCeOwdY
Z4/zjXrklwxkvV9yTzzg7c0=
=qPHB
-----END PGP SIGNATURE-----