4 files changed, 17 insertions, 14 deletions
diff --git a/Swiften/IDN/PlatformIDNConverter.cpp b/Swiften/IDN/PlatformIDNConverter.cpp
index c6104fb..3564bb9 100644
--- a/Swiften/IDN/PlatformIDNConverter.cpp
+++ b/Swiften/IDN/PlatformIDNConverter.cpp
@@ -1,30 +1,30 @@
 /*
- * Copyright (c) 2012-2018 Isode Limited.
+ * Copyright (c) 2012-2019 Isode Limited.
  * All rights reserved.
  * See the COPYING file for more information.
  */
 
 #include <Swiften/IDN/PlatformIDNConverter.h>
 #if defined(HAVE_LIBIDN)
 #include <Swiften/IDN/LibIDNConverter.h>
 #elif defined(HAVE_ICU)
 #include <Swiften/IDN/ICUConverter.h>
 #endif
 
 namespace Swift {
 
-IDNConverter* PlatformIDNConverter::create() {
+std::unique_ptr<IDNConverter> PlatformIDNConverter::create() {
 #if defined(HAVE_LIBIDN)
-    return new LibIDNConverter();
+    return std::make_unique<LibIDNConverter>();
 #elif defined(HAVE_ICU)
-    return new ICUConverter();
+    return std::make_unique<ICUConverter>();
 #else
 #if defined(NEED_IDN)
 #error "No IDN implementation"
 #else
-    return nullptr;
+    return {};
 #endif
 #endif
 }
 
 }
diff --git a/Swiften/IDN/PlatformIDNConverter.h b/Swiften/IDN/PlatformIDNConverter.h
index 704f7a7..3b9a275 100644
--- a/Swiften/IDN/PlatformIDNConverter.h
+++ b/Swiften/IDN/PlatformIDNConverter.h
@@ -1,17 +1,19 @@
 /*
- * Copyright (c) 2012 Isode Limited.
+ * Copyright (c) 2012-2019 Isode Limited.
  * All rights reserved.
  * See the COPYING file for more information.
  */
 
 #pragma once
 
 #include <Swiften/Base/API.h>
 
+#include <memory>
+
 namespace Swift {
     class IDNConverter;
 
     namespace PlatformIDNConverter {
-        SWIFTEN_API IDNConverter* create();
+        SWIFTEN_API std::unique_ptr<IDNConverter> create();
     }
 }
diff --git a/Swiften/Network/BoostNetworkFactories.cpp b/Swiften/Network/BoostNetworkFactories.cpp
index 9fef73c..13a7960 100644
--- a/Swiften/Network/BoostNetworkFactories.cpp
+++ b/Swiften/Network/BoostNetworkFactories.cpp
@@ -1,11 +1,11 @@
 /*
- * Copyright (c) 2010-2016 Isode Limited.
+ * Copyright (c) 2010-2019 Isode Limited.
  * All rights reserved.
  * See the COPYING file for more information.
  */
 
 #include <Swiften/Network/BoostNetworkFactories.h>
 
 #include <Swiften/Crypto/CryptoProvider.h>
 #include <Swiften/Crypto/PlatformCryptoProvider.h>
 #include <Swiften/IDN/IDNConverter.h>
@@ -38,29 +38,28 @@ BoostNetworkFactories::BoostNetworkFactories(EventLoop* eventLoop, std::shared_p
     natTraverser = new NullNATTraverser(eventLoop);
 #endif
     networkEnvironment = new PlatformNetworkEnvironment();
     xmlParserFactory = new PlatformXMLParserFactory();
     tlsFactories = new PlatformTLSFactories();
     proxyProvider = new PlatformProxyProvider();
     idnConverter = PlatformIDNConverter::create();
 #ifdef USE_UNBOUND
     // TODO: What to do about idnConverter.
-    domainNameResolver = new UnboundDomainNameResolver(idnConverter, ioServiceThread.getIOService(), eventLoop);
+    domainNameResolver = new UnboundDomainNameResolver(idnConverter.get(), ioServiceThread.getIOService(), eventLoop);
 #else
-    domainNameResolver = new PlatformDomainNameResolver(idnConverter, eventLoop);
+    domainNameResolver = new PlatformDomainNameResolver(idnConverter.get(), eventLoop);
 #endif
     cryptoProvider = PlatformCryptoProvider::create();
 }
 
 BoostNetworkFactories::~BoostNetworkFactories() {
     delete cryptoProvider;
     delete domainNameResolver;
-    delete idnConverter;
     delete proxyProvider;
     delete tlsFactories;
     delete xmlParserFactory;
     delete networkEnvironment;
     delete natTraverser;
     delete connectionServerFactory;
     delete connectionFactory;
     delete timerFactory;
 }
diff --git a/Swiften/Network/BoostNetworkFactories.h b/Swiften/Network/BoostNetworkFactories.h
index 2bf8d33..33a3584 100644
--- a/Swiften/Network/BoostNetworkFactories.h
+++ b/Swiften/Network/BoostNetworkFactories.h
@@ -1,17 +1,19 @@
 /*
- * Copyright (c) 2010-2017 Isode Limited.
+ * Copyright (c) 2010-2019 Isode Limited.
  * All rights reserved.
  * See the COPYING file for more information.
  */
 
 #pragma once
 
+#include <memory>
+
 #include <Swiften/Base/API.h>
 #include <Swiften/Network/BoostIOServiceThread.h>
 #include <Swiften/Network/NetworkFactories.h>
 
 namespace Swift {
     class EventLoop;
     class NATTraverser;
     class PlatformTLSFactories;
 
@@ -62,32 +64,32 @@ namespace Swift {
             virtual ProxyProvider* getProxyProvider() const override {
                 return proxyProvider;
             }
 
             virtual EventLoop* getEventLoop() const override {
                 return eventLoop;
             }
 
             virtual IDNConverter* getIDNConverter() const override {
-                return idnConverter;
+                return idnConverter.get();
             }
 
             virtual CryptoProvider* getCryptoProvider() const override {
                 return cryptoProvider;
             }
 
         private:
             BoostIOServiceThread ioServiceThread;
             TimerFactory* timerFactory;
             ConnectionFactory* connectionFactory;
             DomainNameResolver* domainNameResolver;
             ConnectionServerFactory* connectionServerFactory;
             NATTraverser* natTraverser;
             NetworkEnvironment* networkEnvironment;
             XMLParserFactory* xmlParserFactory;
             PlatformTLSFactories* tlsFactories;
             ProxyProvider* proxyProvider;
             EventLoop* eventLoop;
-            IDNConverter* idnConverter;
+            std::unique_ptr<IDNConverter> idnConverter;
             CryptoProvider* cryptoProvider;
     };
 }