Replace auto_ptr by unique_ptr

Solve also all ambiguous constructor std::unique_ptr(NULL) calls and fix the linting warnings produces after the find-and-replace of (std::)auto_ptr by (std::)unique_ptr.

Author: aroffringa

common/http/OlaHTTPServer.cpp

@@ -30,7 +30,7 @@ namespace ola {
 namespace http {
 
 using ola::ExportMap;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::ostringstream;
 using std::string;
 using std::vector;
@@ -71,7 +71,7 @@ bool OlaHTTPServer::Init() {
  */
 int OlaHTTPServer::DisplayDebug(const HTTPRequest*,
                                 HTTPResponse *raw_response) {
-  auto_ptr<HTTPResponse> response(raw_response);
+  unique_ptr<HTTPResponse> response(raw_response);
   ola::TimeStamp now;
   m_clock.CurrentMonotonicTime(&now);
   ola::TimeInterval diff = now - m_start_time;
@@ -98,7 +98,7 @@ int OlaHTTPServer::DisplayDebug(const HTTPRequest*,
  */
 int OlaHTTPServer::DisplayHandlers(const HTTPRequest*,
                                    HTTPResponse *raw_response) {
-  auto_ptr<HTTPResponse> response(raw_response);
+  unique_ptr<HTTPResponse> response(raw_response);
   vector<string> handlers;
   m_server.Handlers(&handlers);
   vector<string>::const_iterator iter;

common/io/IOQueueTest.cpp

@@ -31,7 +31,7 @@
 using ola::io::IOQueue;
 using ola::io::IOVec;
 using ola::io::MemoryBlockPool;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::string;
 
 
@@ -59,7 +59,7 @@ class IOQueueTest: public CppUnit::TestFixture {
     void testStringRead();
 
  private:
-    auto_ptr<IOQueue> m_buffer;
+    unique_ptr<IOQueue> m_buffer;
 
     unsigned int SumLengthOfIOVec(const struct IOVec *iov, int iocnt);
 };

common/io/OutputStreamTest.cpp

@@ -32,7 +32,7 @@
 using ola::io::IOQueue;
 using ola::io::BigEndianOutputStream;
 using ola::network::HostToNetwork;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::string;
 
 

common/io/SelectServerTest.cpp

@@ -53,7 +53,7 @@ using ola::io::SelectServer;
 using ola::io::UnixSocket;
 using ola::io::WriteFileDescriptor;
 using ola::network::UDPSocket;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::set;
 
 /*
@@ -543,7 +543,7 @@ void SelectServerTest::testReadWriteInteraction() {
   m_ss->RemoveWriteDescriptor(&socket);
 
   // now the Write end closes
-  auto_ptr<UnixSocket> other_end(socket.OppositeEnd());
+  unique_ptr<UnixSocket> other_end(socket.OppositeEnd());
   other_end->CloseClient();
 
   m_ss->RegisterSingleTimeout(

common/network/AdvancedTCPConnectorTest.cpp

@@ -47,7 +47,7 @@ using ola::network::IPV4Address;
 using ola::network::IPV4SocketAddress;
 using ola::network::TCPAcceptingSocket;
 using ola::network::TCPSocket;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::string;
 
 // used to set a timeout which aborts the tests
@@ -90,7 +90,7 @@ class AdvancedTCPConnectorTest: public CppUnit::TestFixture {
  private:
   ola::MockClock m_clock;
   SelectServer *m_ss;
-  auto_ptr<ola::network::TCPSocketFactory> m_tcp_socket_factory;
+  unique_ptr<ola::network::TCPSocketFactory> m_tcp_socket_factory;
   IPV4Address m_localhost;
   IPV4SocketAddress m_server_address;
   ola::thread::timeout_id m_timeout_id;

common/network/IPV4AddressTest.cpp

@@ -40,7 +40,7 @@
 
 using ola::network::IPV4Address;
 using ola::network::HostToNetwork;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::string;
 using std::vector;
 
@@ -97,11 +97,11 @@ void IPAddressTest::testIPV4Address() {
   OLA_ASSERT_EQ(string("192.168.1.1"), str.str());
 
   // test from string
-  auto_ptr<IPV4Address> string_address(IPV4Address::FromString("10.0.0.1"));
+  unique_ptr<IPV4Address> string_address(IPV4Address::FromString("10.0.0.1"));
   OLA_ASSERT_NOT_NULL(string_address.get());
   OLA_ASSERT_EQ(string("10.0.0.1"), string_address->ToString());
 
-  auto_ptr<IPV4Address> string_address2(IPV4Address::FromString("foo"));
+  unique_ptr<IPV4Address> string_address2(IPV4Address::FromString("foo"));
   OLA_ASSERT_NULL(string_address2.get());
 
   // and the second form

common/network/InterfacePickerTest.cpp

@@ -36,7 +36,7 @@ using ola::network::IPV4Address;
 using ola::network::Interface;
 using ola::network::InterfacePicker;
 using ola::network::MACAddress;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::cout;
 using std::endl;
 using std::string;
@@ -62,7 +62,7 @@ CPPUNIT_TEST_SUITE_REGISTRATION(InterfacePickerTest);
  * Check that we find at least one candidate interface.
  */
 void InterfacePickerTest::testGetInterfaces() {
-  auto_ptr<InterfacePicker> picker(InterfacePicker::NewPicker());
+  unique_ptr<InterfacePicker> picker(InterfacePicker::NewPicker());
   vector<Interface> interfaces = picker->GetInterfaces(true);
 #ifndef _WIN32
   // If a Windows box is not on a network, and doesn't have it's loopback, there
@@ -86,7 +86,7 @@ void InterfacePickerTest::testGetInterfaces() {
  * Check that we find a loopback interface.
  */
 void InterfacePickerTest::testGetLoopbackInterfaces() {
-  auto_ptr<InterfacePicker> picker(InterfacePicker::NewPicker());
+  unique_ptr<InterfacePicker> picker(InterfacePicker::NewPicker());
   vector<Interface> interfaces = picker->GetInterfaces(true);
 #ifndef _WIN32
   // If a Windows box is not on a network, and doesn't have it's loopback, there

common/network/MACAddressTest.cpp

@@ -30,7 +30,7 @@
 #include "ola/testing/TestUtils.h"
 
 using ola::network::MACAddress;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::string;
 using std::vector;
 
@@ -76,17 +76,17 @@ void MACAddressTest::testMACAddress() {
   OLA_ASSERT_EQ(string("01:23:45:67:89:ab"), str.str());
 
   // test from string
-  auto_ptr<MACAddress> string_address(
+  unique_ptr<MACAddress> string_address(
       MACAddress::FromString("fe:dc:ba:98:76:54"));
   OLA_ASSERT_NOT_NULL(string_address.get());
   OLA_ASSERT_EQ(string("fe:dc:ba:98:76:54"), string_address->ToString());
 
-  auto_ptr<MACAddress> string_address2(
+  unique_ptr<MACAddress> string_address2(
       MACAddress::FromString("98.76.54.fe.dc.ba"));
   OLA_ASSERT_NOT_NULL(string_address2.get());
   OLA_ASSERT_EQ(string("98:76:54:fe:dc:ba"), string_address2->ToString());
 
-  auto_ptr<MACAddress> string_address3(MACAddress::FromString("foo"));
+  unique_ptr<MACAddress> string_address3(MACAddress::FromString("foo"));
   OLA_ASSERT_NULL(string_address3.get());
 
   // and the second form

common/network/NetworkUtils.cpp

@@ -586,7 +586,7 @@ static bool GetDefaultRouteWithNetlink(int32_t *if_index,
     return false;
   }
 
-  std::auto_ptr<NetlinkCallback> cb(
+  std::unique_ptr<NetlinkCallback> cb(
       ola::NewCallback(MessageHandler, if_index, default_gateway));
   if (!ReadNetlinkSocket(sd, msg, BUFSIZE, nl_msg->nlmsg_seq, cb.get())) {
     return false;

common/network/TCPConnectorTest.cpp

@@ -45,7 +45,7 @@ using ola::io::SelectServer;
 using ola::network::TCPConnector;
 using ola::network::TCPAcceptingSocket;
 using ola::network::TCPSocket;
-using std::auto_ptr;
+using std::unique_ptr;
 using std::string;
 
 // used to set a timeout which aborts the tests