nsapi - Moved literal ip parsing out of dns-query

- Not inherently a dns operation
- Able to reuse SocketAddress provided to the NetworkInterface

Author: geky

features/net/network-socket/NetworkStack.cpp

@@ -24,11 +24,25 @@
 // Default NetworkStack operations
 int NetworkStack::gethostbyname(const char *name, SocketAddress *address)
 {
+    // check for simple ip addresses
+    if (address->set_ip_address(name)) {
+        return 0;
+    }
+
     return nsapi_dns_query(this, name, address);
 }
 
 int NetworkStack::gethostbyname(const char *name, SocketAddress *address, nsapi_version_t version)
 {
+    // check for simple ip addresses
+    if (address->set_ip_address(name)) {
+        if (address->get_ip_version() != version) {
+            return NSAPI_ERROR_DNS_FAILURE;
+        }
+
+        return 0;
+    }
+
     return nsapi_dns_query(this, name, address, version);
 }
 

features/net/network-socket/nsapi_dns.cpp

@@ -199,17 +199,6 @@ static int nsapi_dns_query_multiple(NetworkStack *stack, const char *host,
         return NSAPI_ERROR_PARAMETER;
     }
 
-    // check for simple ip addresses
-    SocketAddress address;
-    if (address.set_ip_address(host)) {
-        if (address.get_ip_version() != version) {
-            return NSAPI_ERROR_DNS_FAILURE;
-        }
-
-        *addr = address.get_addr();
-        return 0;
-    }
-
     // create a udp socket
     UDPSocket socket;
     int err = socket.open(stack);