Committed by
Gerrit Code Review
Remove unnecessary return in void method
Change-Id: I4863119ae13587f4d7fc60e55094c9d8d701c4e7
Showing
3 changed files
with
0 additions
and
6 deletions
| ... | @@ -410,8 +410,6 @@ public class NetconfDeviceProvider extends AbstractProvider | ... | @@ -410,8 +410,6 @@ public class NetconfDeviceProvider extends AbstractProvider |
| 410 | if (isReachable(deviceId)) { | 410 | if (isReachable(deviceId)) { |
| 411 | controller.connectDevice(deviceId); | 411 | controller.connectDevice(deviceId); |
| 412 | providerService.receivedRoleReply(deviceId, newRole, MastershipRole.MASTER); | 412 | providerService.receivedRoleReply(deviceId, newRole, MastershipRole.MASTER); |
| 413 | - } else { | ||
| 414 | - return; | ||
| 415 | } | 413 | } |
| 416 | } catch (Exception e) { | 414 | } catch (Exception e) { |
| 417 | if (deviceService.getDevice(deviceId) != null) { | 415 | if (deviceService.getDevice(deviceId) != null) { | ... | ... |
providers/ovsdb/device/src/main/java/org/onosproject/ovsdb/providers/device/OvsdbDeviceProvider.java
| ... | @@ -94,7 +94,6 @@ public class OvsdbDeviceProvider extends AbstractProvider | ... | @@ -94,7 +94,6 @@ public class OvsdbDeviceProvider extends AbstractProvider |
| 94 | if (!isReachable(deviceId)) { | 94 | if (!isReachable(deviceId)) { |
| 95 | log.error("Failed to probe device {}", deviceId); | 95 | log.error("Failed to probe device {}", deviceId); |
| 96 | providerService.deviceDisconnected(deviceId); | 96 | providerService.deviceDisconnected(deviceId); |
| 97 | - return; | ||
| 98 | } else { | 97 | } else { |
| 99 | log.trace("Confirmed device {} connection", deviceId); | 98 | log.trace("Confirmed device {} connection", deviceId); |
| 100 | } | 99 | } | ... | ... |
| ... | @@ -113,14 +113,11 @@ public class PcepPacketProvider extends AbstractProvider implements PacketProvid | ... | @@ -113,14 +113,11 @@ public class PcepPacketProvider extends AbstractProvider implements PacketProvid |
| 113 | @Override | 113 | @Override |
| 114 | public void send() { | 114 | public void send() { |
| 115 | // We don't send anything out. | 115 | // We don't send anything out. |
| 116 | - return; | ||
| 117 | } | 116 | } |
| 118 | } | 117 | } |
| 119 | 118 | ||
| 120 | @Override | 119 | @Override |
| 121 | public void emit(OutboundPacket packet) { | 120 | public void emit(OutboundPacket packet) { |
| 122 | // Nothing to emit | 121 | // Nothing to emit |
| 123 | - return; | ||
| 124 | - | ||
| 125 | } | 122 | } |
| 126 | } | 123 | } | ... | ... |
-
Please register or login to post a comment