2011/3/7 <ma...@apache.org>:
> Author: markt
> Date: Mon Mar 7 17:42:52 2011
> New Revision: 1078863
>
> URL: http://svn.apache.org/viewvc?rev=1078863&view=rev
> Log:
> Fix some warnings
>
> Modified:
> tomcat/trunk/java/org/apache/catalina/core/StandardWrapper.java
> tomcat/trunk/java/org/apache/catalina/core/StandardWrapperFacade.java
>
> --- tomcat/trunk/java/org/apache/catalina/core/StandardWrapper.java (original)
> +++ tomcat/trunk/java/org/apache/catalina/core/StandardWrapper.java Mon Mar
> 7 17:42:52 2011
> @@ -78,6 +78,7 @@ import org.apache.tomcat.util.modeler.Re
> * @author Remy Maucherat
> * @version $Id$
> */
> +@SuppressWarnings("deprecation") // SingleThreadModel
> public class StandardWrapper extends ContainerBase
> implements ServletConfig, Wrapper, NotificationEmitter {
>
> @@ -1602,8 +1603,7 @@ public class StandardWrapper extends Con
> *
> * @param mapperClass Java class name of the default Mapper
> */
> - protected void addDefaultMapper(
> - @SuppressWarnings("unused") String mapperClass) {
> + protected void addDefaultMapper(String mapperClass) {
>
> // No need for a default Mapper on a Wrapper
>
It looks like the StandardWrapper.addDefaultMapper() method can be
removed as a whole.
(A protected method that is never called, does not @Override anything,
and does nothing).
Best regards,
Konstantin Kolinko
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
