Bug 303606

Description Michael Moser 2010-02-23 06:47:01 EST

Build Identifier: Build id: 20090621-0832

I have a type where some method declares an exception to allow derived types to throw that exception. The basic implemenation however just returns null and thus does not actually throw any exception. 

However, when I silence the resulting warning using a @SuppressWarning("unused") annotation then I get another warning stating "Unnecessary @SuppressWarning("unused")". The latter is probably due to the fact, that the compiler recognices, that the throws is NOT unused and hence the annotation is superfluous. 

I append a condensed example that shows the problem below.
How do I phrase this code without getting *any* warning?

Michael

-----------------------------------------
package test;

public class Test
{
	@SuppressWarnings("serial")
	class MyException extends Exception {
		MyException(String msg) {
			super(msg);
		}
	}
	
	class A {
		protected Object foo(Object x) throws MyException {
			return null;
		}
	}

	class B extends A {
		@Override
		protected Object foo(Object x) throws MyException {
			if (x == null) throw new MyException("x is null");
			return x.getClass();
		}
	}
}
-----------------------------------------

Reproducible: Always