Reimplements net::AddressList without struct addrinfo.

net/socket/tcp_client_socket_libevent.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.
 
 #ifndef NET_SOCKET_TCP_CLIENT_SOCKET_LIBEVENT_H_
 #define NET_SOCKET_TCP_CLIENT_SOCKET_LIBEVENT_H_
 #pragma once
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 137 matching lines @@
   // Local IP address and port we are bound to. Set to NULL if Bind()
   // was't called (in that cases OS chooses address/port).
   scoped_ptr<IPEndPoint> bind_address_;
 
   // Stores bound socket between Bind() and Connect() calls.
   int bound_socket_;
 
   // The list of addresses we should try in order to establish a connection.
   AddressList addresses_;
 
-  // Where we are in above list, or NULL if all addrinfos have been tried.
-  const struct addrinfo* current_ai_;
+  // Where we are in above list. Set to -1 if uninitialized.

[eroman, 2012/05/04 01:08:41]

I don't like -1 for unsigned values. Please make this an int.

[szym, 2012/05/04 02:38:29]

Will do. That was my initial attempt, but then go lazy with static_cast<int>()
for all DCHECKs and comparisons.

+  size_t current_address_index_;
 
   // The socket's libevent wrappers
   MessageLoopForIO::FileDescriptorWatcher read_socket_watcher_;
   MessageLoopForIO::FileDescriptorWatcher write_socket_watcher_;
 
   // The corresponding watchers for reads and writes.
   ReadWatcher read_watcher_;
   WriteWatcher write_watcher_;
 
   // The buffer used by OnSocketReady to retry Read requests
@@ skipping 34 matching lines @@
   base::TimeTicks connect_start_time_;
   base::TimeDelta connect_time_micros_;
   int64 num_bytes_read_;
 
   DISALLOW_COPY_AND_ASSIGN(TCPClientSocketLibevent);
 };
 
 }  // namespace net
 
 #endif  // NET_SOCKET_TCP_CLIENT_SOCKET_LIBEVENT_H_