Skip to content
Snippets Groups Projects
Commit 52d14497 authored by Lucas Chiesa's avatar Lucas Chiesa Committed by Daniel Stonier
Browse files

Removed listAllInetAddress private method.

Also renamed getAllInetAddressByName to getAllInetAddressesByName.
parent d02c9244
Branches
Tags
No related merge requests found
...@@ -44,7 +44,13 @@ public class InetAddressFactory { ...@@ -44,7 +44,13 @@ public class InetAddressFactory {
return address.getAddress().length == 4; return address.getAddress().length == 4;
} }
private static List<InetAddress> listAllInetAddress (Collection<NetworkInterface> networkInterfaces) { private static Collection<InetAddress> getAllInetAddresses() {
List<NetworkInterface> networkInterfaces;
try {
networkInterfaces = Collections.list(NetworkInterface.getNetworkInterfaces());
} catch (SocketException e) {
throw new RosRuntimeException(e);
}
List<InetAddress> inetAddresses = Lists.newArrayList(); List<InetAddress> inetAddresses = Lists.newArrayList();
for (NetworkInterface networkInterface : networkInterfaces) { for (NetworkInterface networkInterface : networkInterfaces) {
try { try {
...@@ -58,16 +64,6 @@ public class InetAddressFactory { ...@@ -58,16 +64,6 @@ public class InetAddressFactory {
return inetAddresses; return inetAddresses;
} }
private static Collection<InetAddress> getAllInetAddresses() {
List<NetworkInterface> networkInterfaces;
try {
networkInterfaces = Collections.list(NetworkInterface.getNetworkInterfaces());
} catch (SocketException e) {
throw new RosRuntimeException(e);
}
return listAllInetAddress(networkInterfaces);
}
private static InetAddress filterInetAddresses (Collection<InetAddress> inetAddresses) { private static InetAddress filterInetAddresses (Collection<InetAddress> inetAddresses) {
for (InetAddress address : inetAddresses) { for (InetAddress address : inetAddresses) {
// IPv4 only for now. // IPv4 only for now.
...@@ -86,7 +82,7 @@ public class InetAddressFactory { ...@@ -86,7 +82,7 @@ public class InetAddressFactory {
return filterInetAddresses(Collections.list(networkInterface.getInetAddresses())); return filterInetAddresses(Collections.list(networkInterface.getInetAddresses()));
} }
private static Collection<InetAddress> getAllInetAddressByName(String host) { private static Collection<InetAddress> getAllInetAddressesByName(String host) {
InetAddress[] allAddressesByName; InetAddress[] allAddressesByName;
try { try {
allAddressesByName = org.xbill.DNS.Address.getAllByName(host); allAddressesByName = org.xbill.DNS.Address.getAllByName(host);
...@@ -130,7 +126,7 @@ public class InetAddressFactory { ...@@ -130,7 +126,7 @@ public class InetAddressFactory {
} catch (UnknownHostException e) { } catch (UnknownHostException e) {
throw new RosRuntimeException(e); throw new RosRuntimeException(e);
} }
Collection<InetAddress> allAddressesByName = getAllInetAddressByName(host); Collection<InetAddress> allAddressesByName = getAllInetAddressesByName(host);
// First, try to find a non-loopback IPv4 address. // First, try to find a non-loopback IPv4 address.
for (InetAddress address : allAddressesByName) { for (InetAddress address : allAddressesByName) {
if (!address.isLoopbackAddress() && isIpv4(address)) { if (!address.isLoopbackAddress() && isIpv4(address)) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment