Details
-
Improvement
-
Status: Resolved
-
Minor
-
Resolution: Fixed
-
None
-
None
Description
There are some instances of the following code:
final Channel channel = this.key.channel();
if (channel.isOpen()) {
try
catch (final IOException ignore) {}
}
Eclipse generates a warning that the channel might not be closed.
Obviously that is a false positive, but why bother checking the state?
Surely channel.close() can be repeated without harm?