Change PPB_NetAddress_Private.GetFamily to return a PP_AddressFamily_Private value instead of uint1…

ppapi/c/private/ppb_net_address_private.h

 /* Copyright (c) 2012 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 /* From private/ppb_net_address_private.idl,
- *   modified Sun Feb  5 10:36:30 2012.
+ *   modified Tue Feb 14 14:02:57 2012.
  */
 
 #ifndef PPAPI_C_PRIVATE_PPB_NET_ADDRESS_PRIVATE_H_
 #define PPAPI_C_PRIVATE_PPB_NET_ADDRESS_PRIVATE_H_
 
 #include "ppapi/c/pp_bool.h"
 #include "ppapi/c/pp_macros.h"
 #include "ppapi/c/pp_module.h"
 #include "ppapi/c/pp_stdint.h"
 #include "ppapi/c/pp_var.h"
 
 #define PPB_NETADDRESS_PRIVATE_INTERFACE_0_1 "PPB_NetAddress_Private;0.1"
 #define PPB_NETADDRESS_PRIVATE_INTERFACE_1_0 "PPB_NetAddress_Private;1.0"
 #define PPB_NETADDRESS_PRIVATE_INTERFACE PPB_NETADDRESS_PRIVATE_INTERFACE_1_0
 
 /**
  * @file
  * This file defines the <code>PPB_NetAddress_Private</code> interface.
  */
 
 
 /**
+ * @addtogroup Enums
+ * @{
+ */
+typedef enum {
+  /**
+   * The address family is unspecified.
+   */
+  PP_ADDRESSFAMILY_UNSPECIFIED = 0,
+  /**
+   * The Internet Protocol version 4 (IPv4) address family.
+   */
+  PP_ADDRESSFAMILY_IPV4 = 1,
+  /**
+   * The Internet Protocol version 6 (IPv6) address family.
+   */
+  PP_ADDRESSFAMILY_IPV6 = 2
+} PP_AddressFamily_Private;

[viettrungluu, 2012/02/15 01:43:03]

I'm not sure I like this name a lot. (I might have called it
PP_NetAddress_Family_Private -- or is that PP_NetAddress_Private_Family?) (The
point being that it associates it with "NetAddress".) But whatever.

[yzshen1, 2012/02/15 18:40:55]

I did the change, but without the '_' between NetAddress and Family, so that it
is more consistent with others.

On 2012/02/15 01:43:03, viettrungluu wrote:

+PP_COMPILE_ASSERT_SIZE_IN_BYTES(PP_AddressFamily_Private, 4);
+/**
+ * @}
+ */
+
+/**
  * @addtogroup Structs
  * @{
  */
 /**
  * This is an opaque type holding a network address.
  */
 struct PP_NetAddress_Private {
   uint32_t size;
   char data[128];
 };
@@ skipping 35 matching lines @@
   PP_Bool (*ReplacePort)(const struct PP_NetAddress_Private* src_addr,
                          uint16_t port,
                          struct PP_NetAddress_Private* dest_addr);
   /**
    * Gets the "any" address (for IPv4 or IPv6); for use with UDP Bind.
    */
   void (*GetAnyAddress)(PP_Bool is_ipv6, struct PP_NetAddress_Private* addr);
   /**
    * Gets the address family.
    */
-  uint16_t (*GetFamily)(const struct PP_NetAddress_Private* addr);
+  PP_AddressFamily_Private (*GetFamily)(
+      const struct PP_NetAddress_Private* addr);
   /**
    * Gets the port. The port is returned in host byte order.
    */
   uint16_t (*GetPort)(const struct PP_NetAddress_Private* addr);
   /**
    * Gets the address. The output, address, must be large enough for the
    * current socket family. The output will be the binary representation of an
    * address for the current socket family. For IPv4 and IPv6 the address is in
    * network byte order. PP_TRUE is returned if the address was successfully
    * retrieved.
@@ skipping 17 matching lines @@
                          uint16_t port,
                          struct PP_NetAddress_Private* dest_addr);
   void (*GetAnyAddress)(PP_Bool is_ipv6, struct PP_NetAddress_Private* addr);
 };
 /**
  * @}
  */
 
 #endif  /* PPAPI_C_PRIVATE_PPB_NET_ADDRESS_PRIVATE_H_ */