On Wed, 2009-02-25 at 15:51 +0000, sebb wrote: > On 25/02/2009, Oleg Kalnichevski <[email protected]> wrote: > > On Wed, 2009-02-25 at 13:20 +0000, sebb wrote: > > > On 25/02/2009, Oleg Kalnichevski <[email protected]> wrote: > > > > sebb wrote: > > > > > > > > > There are a few Eclipse warnings of unthrown Exceptions in NIO, for > > > > example: > > > > > > > > > > SSLIOSession.inboundTransport() throws IOException > > > > > > > > > > which does not actually throw any Exceptions. > > > > > > > > > > Should these be removed? > > > > > Documented? > > > > > @Suppressed? > > > > > > > > > > > > > > > > > > I do not think 'throws' declaration can be removed without breaking > > API > > > > compatibility. One possibility could be to copy the method, remove > > 'throws', > > > > change the original method to call the new one, and deprecate the > > original > > > > method. I am not sure it is worth the trouble, so @Suppressed should > > be the > > > > easiest work-around. > > > > > > Or adding @throws to the Javadoc might be better, as one can document > > > that it is not actually thrown, e.g. > > > > > > * @throws IOException - not thrown currently > > > > > > > > > True, but this will not solve the problem with Eclipse, will it? > > > > Actually, it does. >
Cool. Then, it is certainly the best thing to do. Oleg > > Oleg > > > > > > > > > > Cheers > > > > > > > > Oleg > > > > > > > > > > > > > S/// > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > To unsubscribe, e-mail: [email protected] > > > > > For additional commands, e-mail: [email protected] > > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: [email protected] > > > > For additional commands, e-mail: [email protected] > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [email protected] > > > For additional commands, e-mail: [email protected] > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
