Cleanup ReconnectPromise a bit 87/97387/5
authorRobert Varga <robert.varga@pantheon.tech>
Sat, 4 Sep 2021 00:48:41 +0000 (02:48 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Sat, 4 Sep 2021 01:11:38 +0000 (03:11 +0200)
commit82f3e83d6e777a5e76b8b6bca5a8c2646a4b82be
tree296c5d4755fc0214ea5946ed9e6c0c85580df63e
parent23e591d6034f2817b6f32902c621fdf8e0b5c8e4
Cleanup ReconnectPromise a bit

ClosedChannelHandler is really a stateless object used by
ReconnectPromise wiring. Acknowledge that fact by removing the class in
favor of a field initialized by an anonoymous inner class.

While we are here, also document locking rules and generally tighten
things up, eliminating a SpotBugs suppression in the process.

Change-Id: I6abc0ddf296dd9e2f5ea6de5a709003cf14edc7a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
netconf/netconf-netty-util/src/main/java/org/opendaylight/netconf/nettyutil/ReconnectPromise.java