Get rid of some Guava dependencies 38/80138/4
authorJakub Morvay <jmorvay@frinx.io>
Mon, 4 Feb 2019 20:01:11 +0000 (21:01 +0100)
committerJakub Morvay <jmorvay@frinx.io>
Tue, 5 Feb 2019 09:39:04 +0000 (10:39 +0100)
Use Java 8 Optional and Objects requireNonNull instead of their guava
counterparts.

Change-Id: Ieaebd36ec8b3fc9745739c243b3ee8285de8a04b
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>
netconf/netconf-topology-singleton/src/main/java/org/opendaylight/netconf/topology/singleton/impl/RemoteDeviceConnectorImpl.java
netconf/netconf-topology/src/main/java/org/opendaylight/netconf/topology/AbstractNetconfTopology.java

index 6c05cf1fd53b190b2cb55f61a02b6ca0a75be65a..a85e6d6776e6fb8c59834f55ca8cea65644306e5 100644 (file)
@@ -8,6 +8,9 @@
 
 package org.opendaylight.netconf.topology.singleton.impl;
 
+import static java.util.Objects.isNull;
+import static java.util.Objects.requireNonNull;
+
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
 import com.google.common.collect.Lists;
@@ -21,7 +24,6 @@ import java.net.URL;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
-import java.util.Objects;
 import java.util.Optional;
 import javax.annotation.Nullable;
 import org.opendaylight.aaa.encrypt.AAAEncryptionService;
@@ -94,7 +96,7 @@ public class RemoteDeviceConnectorImpl implements RemoteDeviceConnector {
     public RemoteDeviceConnectorImpl(final NetconfTopologySetup netconfTopologyDeviceSetup,
                                      final RemoteDeviceId remoteDeviceId) {
 
-        this.netconfTopologyDeviceSetup = Preconditions.checkNotNull(netconfTopologyDeviceSetup);
+        this.netconfTopologyDeviceSetup = requireNonNull(netconfTopologyDeviceSetup);
         this.remoteDeviceId = remoteDeviceId;
         this.privateKeyPath = netconfTopologyDeviceSetup.getPrivateKeyPath();
         this.privateKeyPassphrase = netconfTopologyDeviceSetup.getPrivateKeyPassphrase();
@@ -107,8 +109,8 @@ public class RemoteDeviceConnectorImpl implements RemoteDeviceConnector {
 
         final NetconfNode netconfNode = netconfTopologyDeviceSetup.getNode().augmentation(NetconfNode.class);
         final NodeId nodeId = netconfTopologyDeviceSetup.getNode().getNodeId();
-        Preconditions.checkNotNull(netconfNode.getHost());
-        Preconditions.checkNotNull(netconfNode.getPort());
+        requireNonNull(netconfNode.getHost());
+        requireNonNull(netconfNode.getPort());
 
         this.deviceCommunicatorDTO = createDeviceCommunicator(nodeId, netconfNode, deviceHandler);
         final NetconfDeviceCommunicator deviceCommunicator = deviceCommunicatorDTO.getCommunicator();
@@ -217,9 +219,9 @@ public class RemoteDeviceConnectorImpl implements RemoteDeviceConnector {
         NetconfDeviceCommunicator netconfDeviceCommunicator =
              userCapabilities.isPresent() ? new NetconfDeviceCommunicator(remoteDeviceId, device,
              new UserPreferences(userCapabilities.get(),
-                 Objects.isNull(node.getYangModuleCapabilities())
+                 isNull(node.getYangModuleCapabilities())
                          ? false : node.getYangModuleCapabilities().isOverride(),
-                 Objects.isNull(node.getNonModuleCapabilities())
+                 isNull(node.getNonModuleCapabilities())
                          ? false : node.getNonModuleCapabilities().isOverride()), rpcMessageLimit)
             : new NetconfDeviceCommunicator(remoteDeviceId, device, rpcMessageLimit);
 
index 3bb3fcf68250099a32dc5595499d5cf31a7d7a38..446dd150a50b95a5df0ab2dee627a399c44b1909 100644 (file)
@@ -7,7 +7,8 @@
  */
 package org.opendaylight.netconf.topology;
 
-import com.google.common.base.Optional;
+import static java.util.Objects.requireNonNull;
+
 import com.google.common.base.Preconditions;
 import com.google.common.base.Strings;
 import com.google.common.collect.Lists;
@@ -26,6 +27,7 @@ import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Optional;
 import java.util.concurrent.TimeUnit;
 import org.opendaylight.aaa.encrypt.AAAEncryptionService;
 import org.opendaylight.controller.config.threadpool.ScheduledThreadPool;
@@ -262,8 +264,8 @@ public abstract class AbstractNetconfTopology implements NetconfTopology {
                                                                           final Node configNode) {
         final NetconfNode netconfNode = configNode.augmentation(NetconfNode.class);
 
-        Preconditions.checkNotNull(netconfNode.getHost());
-        Preconditions.checkNotNull(netconfNode.getPort());
+        requireNonNull(netconfNode.getHost());
+        requireNonNull(netconfNode.getPort());
 
         final NetconfConnectorDTO deviceCommunicatorDTO = createDeviceCommunicator(nodeId, netconfNode);
         final NetconfDeviceCommunicator deviceCommunicator = deviceCommunicatorDTO.getCommunicator();
@@ -561,7 +563,7 @@ public abstract class AbstractNetconfTopology implements NetconfTopology {
         // if none of yang-module-capabilities or non-module-capabilities is specified
         // just return absent
         if (node.getYangModuleCapabilities() == null && node.getNonModuleCapabilities() == null) {
-            return Optional.absent();
+            return Optional.empty();
         }
 
         final List<String> capabilities = new ArrayList<>();