Andrew Nguyen : ipconfig: Move socket address string generation to a helper function.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 7df2d4b231bf470291bfb2d1f728d6ec7f436fc4
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=7df2d4b231bf470291bfb2d1f7...

Author: Andrew Nguyen <anguyen(a)codeweavers.com>
Date:   Mon Sep 27 04:30:49 2010 -0500

ipconfig: Move socket address string generation to a helper function.

---

 programs/ipconfig/ipconfig.c |   23 +++++++++++------------
 1 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/programs/ipconfig/ipconfig.c b/programs/ipconfig/ipconfig.c
index 84b0ace..bf186c1 100644
--- a/programs/ipconfig/ipconfig.c
+++ b/programs/ipconfig/ipconfig.c
@@ -130,6 +130,13 @@ static void print_field(int msg, const WCHAR *value)
     ipconfig_printfW(formatW, field, value);
 }
 
+static BOOL socket_address_to_string(WCHAR *buf, DWORD len, SOCKET_ADDRESS *addr)
+{
+    return WSAAddressToStringW(addr->lpSockaddr,
+                               addr->iSockaddrLength, NULL,
+                               buf, &len) == 0;
+}
+
 static void print_basic_information(void)
 {
     IP_ADAPTER_ADDRESSES *adapters;
@@ -150,6 +157,7 @@ static void print_basic_information(void)
                 static const WCHAR newlineW[] = {'\n',0};
 
                 IP_ADAPTER_UNICAST_ADDRESS *addr;
+                WCHAR addr_buf[54];
 
                 ipconfig_message_printfW(STRING_ADAPTER_FRIENDLY, iftype_to_string(p->IfType), p->FriendlyName);
                 ipconfig_printfW(newlineW);
@@ -157,12 +165,7 @@ static void print_basic_information(void)
 
                 for (addr = p->FirstUnicastAddress; addr; addr = addr->Next)
                 {
-                    WCHAR addr_buf[54];
-                    DWORD len = sizeof(addr_buf)/sizeof(WCHAR);
-
-                    if (WSAAddressToStringW(addr->Address.lpSockaddr,
-                                            addr->Address.iSockaddrLength, NULL,
-                                            addr_buf, &len) == 0)
+                    if (socket_address_to_string(addr_buf, sizeof(addr_buf)/sizeof(WCHAR), &addr->Address))
                         print_field(STRING_IP_ADDRESS, addr_buf);
                     /* FIXME: Output corresponding subnet mask. */
                 }
@@ -287,6 +290,7 @@ static void print_full_information(void)
             {
                 IP_ADAPTER_UNICAST_ADDRESS *addr;
                 WCHAR physaddr_buf[3 * MAX_ADAPTER_ADDRESS_LENGTH];
+                WCHAR addr_buf[54];
 
                 ipconfig_message_printfW(STRING_ADAPTER_FRIENDLY, iftype_to_string(p->IfType), p->FriendlyName);
                 ipconfig_printfW(newlineW);
@@ -299,12 +303,7 @@ static void print_full_information(void)
 
                 for (addr = p->FirstUnicastAddress; addr; addr = addr->Next)
                 {
-                    WCHAR addr_buf[54];
-                    DWORD len = sizeof(addr_buf)/sizeof(WCHAR);
-
-                    if (WSAAddressToStringW(addr->Address.lpSockaddr,
-                                            addr->Address.iSockaddrLength, NULL,
-                                            addr_buf, &len) == 0)
+                    if (socket_address_to_string(addr_buf, sizeof(addr_buf)/sizeof(WCHAR), &addr->Address))
                         print_field(STRING_IP_ADDRESS, addr_buf);
                     /* FIXME: Output corresponding subnet mask. */
                 }