Removed duplicate NSString NSURL conversion functionsv5.3.0-alpha1

Currently there's a duplication of the QString <-> NSString and
QUrl <-> NSURL functions.
This patch removes the private implementation and updates the code to
to use QtCore's functions directly.

Change-Id: I8fa613ac6753cc16fedd208f676f39ee2fec6bfa
Reviewed-by: Jake Petroules <jake.petroules@petroules.com>
Reviewed-by: Morten Johan Sørvig <morten.sorvig@digia.com>

6 files changed, 14 insertions, 50 deletions

diff --git a/src/macextras/qmacfunctions.mm b/src/macextras/qmacfunctions.mm
index 187c6ee..37185a6 100644
--- a/src/macextras/qmacfunctions.mm
+++ b/src/macextras/qmacfunctions.mm
@@ -44,6 +44,7 @@
 #include <QByteArray>
 #include <QPixmap>
 #include <QUrl>
+#include <QString>
 
 #include "qmacfunctions_p.h"
 
@@ -65,33 +66,6 @@ QT_BEGIN_NAMESPACE
 namespace QtMac
 {
 
-NSString *toNSString(const QString &string)
-{
-    return [NSString stringWithCharacters:reinterpret_cast<const UniChar*>(string.unicode()) length:string.length()];
-}
-
-QString fromNSString(const NSString *string)
-{
-    if (!string)
-        return QString();
-
-    QString qstring;
-    qstring.resize([string length]);
-    [string getCharacters:reinterpret_cast<unichar*>(qstring.data()) range:NSMakeRange(0, [string length])];
-
-    return qstring;
-}
-
-NSURL* toNSURL(const QUrl &url)
-{
-    return [NSURL URLWithString:toNSString(url.toString())];
-}
-
-QUrl fromNSURL(const NSURL *url)
-{
-    return QUrl(fromNSString([url absoluteString]));
-}
-
 NSData* toNSData(const QByteArray &data)
 {
     return [NSData dataWithBytes:data.constData() length:data.size()];
diff --git a/src/macextras/qmacfunctions_ios.mm b/src/macextras/qmacfunctions_ios.mm
index fa83ce6..4004f5b 100644
--- a/src/macextras/qmacfunctions_ios.mm
+++ b/src/macextras/qmacfunctions_ios.mm
@@ -66,7 +66,7 @@ CGContextRef currentCGContext()
 */
 void setApplicationIconBadgeNumber(int number)
 {
-    [[UIApplication sharedApplication] setApplicationIconBadgeNumber: number];
+    [[UIApplication sharedApplication] setApplicationIconBadgeNumber:number];
 }
 
 /*!
diff --git a/src/macextras/qmacfunctions_mac.mm b/src/macextras/qmacfunctions_mac.mm
index 88ebb61..38f941f 100644
--- a/src/macextras/qmacfunctions_mac.mm
+++ b/src/macextras/qmacfunctions_mac.mm
@@ -45,6 +45,8 @@
 #include "qmacfunctions.h"
 #include "qmacfunctions_p.h"
 
+#include <QtCore/QString>
+
 #if QT_VERSION > QT_VERSION_CHECK(5, 0, 0)
 #include <QtGui/QWindow>
 #include <QtWidgets/QMenu>
@@ -104,7 +106,7 @@ CGContextRef currentCGContext()
 */
 void setBadgeLabelText(const QString &text)
 {
-    [[[NSApplication sharedApplication] dockTile] setBadgeLabel:toNSString(text)];
+    [[[NSApplication sharedApplication] dockTile] setBadgeLabel:text.toNSString()];
 }
 
 /*!
@@ -114,7 +116,7 @@ void setBadgeLabelText(const QString &text)
 */
 QString badgeLabelText()
 {
-    return fromNSString([[[NSApplication sharedApplication] dockTile] badgeLabel]);
+    return QString::fromNSString([[[NSApplication sharedApplication] dockTile] badgeLabel]);
 }
 
 } // namespace QtMac
diff --git a/src/macextras/qmacfunctions_p.h b/src/macextras/qmacfunctions_p.h
index b03b7ae..6d8211b 100644
--- a/src/macextras/qmacfunctions_p.h
+++ b/src/macextras/qmacfunctions_p.h
@@ -49,22 +49,8 @@
 #include <QtGui/QGuiApplication>
 #include <qpa/qplatformnativeinterface.h>
 
-// ### remove when merged to QtCore
-
-Q_FORWARD_DECLARE_OBJC_CLASS(NSString);
-Q_FORWARD_DECLARE_OBJC_CLASS(NSURL);
-
 QT_BEGIN_NAMESPACE
 
-namespace QtMac
-{
-    Q_MACEXTRAS_EXPORT NSString *toNSString(const QString &string);
-    Q_MACEXTRAS_EXPORT QString fromNSString(const NSString *string);
-
-    Q_MACEXTRAS_EXPORT NSURL *toNSURL(const QUrl &url);
-    Q_MACEXTRAS_EXPORT QUrl fromNSURL(const NSURL *url);
-}
-
 inline QPlatformNativeInterface::NativeResourceForIntegrationFunction resolvePlatformFunction(const QByteArray &functionName)
 {
     QPlatformNativeInterface *nativeInterface = QGuiApplication::platformNativeInterface();
diff --git a/src/macextras/qmactoolbar.mm b/src/macextras/qmactoolbar.mm
index 8093920..c75c9e8 100644
--- a/src/macextras/qmactoolbar.mm
+++ b/src/macextras/qmactoolbar.mm
@@ -45,6 +45,7 @@
 #include <QtCore/QDebug>
 #include <QtCore/QTimer>
 #include <QtCore/QUuid>
+#include <QtCore/QString>
 #include <QtCore/qdebug.h>
 
 #if QT_VERSION >= QT_VERSION_CHECK(5, 0, 0)
@@ -324,7 +325,7 @@ NSMutableArray *QMacToolBarPrivate::getItemIdentifiers(const QList<QMacToolBarIt
 
 void QMacToolBarPrivate::itemClicked(NSToolbarItem *item)
 {
-    QString identifier = QtMac::fromNSString([item itemIdentifier]);
+    QString identifier = QString::fromNSString([item itemIdentifier]);
     QMacToolBarItem *toolButton = reinterpret_cast<QMacToolBarItem *>(identifier.toULongLong());
     Q_EMIT toolButton->activated();
 }
diff --git a/src/macextras/qmactoolbardelegate.mm b/src/macextras/qmactoolbardelegate.mm
index b6f3768..804879f 100644
--- a/src/macextras/qmactoolbardelegate.mm
+++ b/src/macextras/qmactoolbardelegate.mm
@@ -44,8 +44,9 @@
 #include <QtGui/QImage>
 #include <QtGui/QPixmap>
 
+#include <QtCore/QString>
+
 #include "qmacfunctions.h"
-#include "qmacfunctions_p.h"
 #include "qmactoolbar.h"
 
 QT_USE_NAMESPACE
@@ -54,7 +55,7 @@ NSArray *toNSArray(const QList<QString> &stringList)
 {
     NSMutableArray *array = [[NSMutableArray alloc] init];
     foreach (const QString &string, stringList) {
-        [array addObject : QtMac::toNSString(string)];
+        [array addObject:string.toNSString()];
     }
     return array;
 }
@@ -107,12 +108,12 @@ QString qt_strippedText(QString s)
 {
     Q_UNUSED(toolbar);
     Q_UNUSED(willBeInserted);
-    const QString identifier = QtMac::fromNSString(itemIdentifier);
+    const QString identifier = QString::fromNSString(itemIdentifier);
     QMacToolBarItem *toolButton = reinterpret_cast<QMacToolBarItem *>(identifier.toULongLong()); // string -> unisgned long long -> pointer
     NSToolbarItem *toolbarItem = toolButton->nativeToolBarItem();
 
-    [toolbarItem setTarget : self];
-    [toolbarItem setAction : @selector(itemClicked:)];
+    [toolbarItem setTarget:self];
+    [toolbarItem setAction:@selector(itemClicked:)];
 
     return toolbarItem;
 }