1 files changed, 19 insertions, 17 deletions

diff --git a/src/corelib/time/qtimezoneprivate_mac.mm b/src/corelib/time/qtimezoneprivate_mac.mm
index 4509e316f9..1fb48a31d3 100644
--- a/src/corelib/time/qtimezoneprivate_mac.mm
+++ b/src/corelib/time/qtimezoneprivate_mac.mm
@@ -60,22 +60,24 @@ QT_BEGIN_NAMESPACE
 
 // Create the system default time zone
 QMacTimeZonePrivate::QMacTimeZonePrivate()
-    : m_nstz(0)
 {
-    init(systemTimeZoneId());
+    // Reset the cached system tz then instantiate it:
+    [NSTimeZone resetSystemTimeZone];
+    m_nstz = [NSTimeZone.systemTimeZone retain];
+    Q_ASSERT(m_nstz);
+    m_id = QString::fromNSString(m_nstz.name).toUtf8();
 }
 
 // Create a named time zone
 QMacTimeZonePrivate::QMacTimeZonePrivate(const QByteArray &ianaId)
-    : m_nstz(0)
+    : m_nstz(nil)
 {
     init(ianaId);
 }
 
 QMacTimeZonePrivate::QMacTimeZonePrivate(const QMacTimeZonePrivate &other)
-    : QTimeZonePrivate(other), m_nstz(0)
+    : QTimeZonePrivate(other), m_nstz([other.m_nstz copy])
 {
-    m_nstz = [other.m_nstz copy];
 }
 
 QMacTimeZonePrivate::~QMacTimeZonePrivate()
@@ -109,7 +111,7 @@ void QMacTimeZonePrivate::init(const QByteArray &ianaId)
 
 QString QMacTimeZonePrivate::comment() const
 {
-    return QString::fromNSString([m_nstz description]);
+    return QString::fromNSString(m_nstz.description);
 }
 
 QString QMacTimeZonePrivate::displayName(QTimeZone::TimeType timeType,
@@ -212,7 +214,7 @@ bool QMacTimeZonePrivate::hasTransitions() const
     // TODO Not sure what is returned in event of no transitions, assume will be before requested date
     NSDate *epoch = [NSDate dateWithTimeIntervalSince1970:0];
     const NSDate *date = [m_nstz nextDaylightSavingTimeTransitionAfterDate:epoch];
-    const bool result = ([date timeIntervalSince1970] > [epoch timeIntervalSince1970]);
+    const bool result = (date.timeIntervalSince1970 > epoch.timeIntervalSince1970);
     return result;
 }
 
@@ -222,7 +224,7 @@ QTimeZonePrivate::Data QMacTimeZonePrivate::nextTransition(qint64 afterMSecsSinc
     const NSTimeInterval seconds = afterMSecsSinceEpoch / 1000.0;
     NSDate *nextDate = [NSDate dateWithTimeIntervalSince1970:seconds];
     nextDate = [m_nstz nextDaylightSavingTimeTransitionAfterDate:nextDate];
-    const NSTimeInterval nextSecs = [nextDate timeIntervalSince1970];
+    const NSTimeInterval nextSecs = nextDate.timeIntervalSince1970;
     if (nextDate == nil || nextSecs <= seconds) {
         [nextDate release];
         return invalidData();
@@ -248,7 +250,7 @@ QTimeZonePrivate::Data QMacTimeZonePrivate::previousTransition(qint64 beforeMSec
     NSDate *nextDate = [NSDate dateWithTimeIntervalSince1970:nextSecs];
     nextDate = [m_nstz nextDaylightSavingTimeTransitionAfterDate:nextDate];
     if (nextDate != nil
-        && (tranSecs = [nextDate timeIntervalSince1970]) < endSecs) {
+        && (tranSecs = nextDate.timeIntervalSince1970) < endSecs) {
         // There's a transition within the last year before endSecs:
         nextSecs = tranSecs;
     } else {
@@ -257,7 +259,7 @@ QTimeZonePrivate::Data QMacTimeZonePrivate::previousTransition(qint64 beforeMSec
         nextDate = [m_nstz nextDaylightSavingTimeTransitionAfterDate:nextDate];
         if (nextDate != nil) {
             NSTimeInterval lateSecs = nextSecs;
-            nextSecs = [nextDate timeIntervalSince1970];
+            nextSecs = nextDate.timeIntervalSince1970;
             Q_ASSERT(nextSecs <= endSecs - year || nextSecs == tranSecs);
             /*
               We're looking at the first ever transition for our zone, at
@@ -283,8 +285,7 @@ QTimeZonePrivate::Data QMacTimeZonePrivate::previousTransition(qint64 beforeMSec
                 NSTimeInterval middle = nextSecs / 2 + lateSecs / 2;
                 NSDate *split = [NSDate dateWithTimeIntervalSince1970:middle];
                 split = [m_nstz nextDaylightSavingTimeTransitionAfterDate:split];
-                if (split != nil
-                    && (tranSecs = [split timeIntervalSince1970]) < endSecs) {
+                if (split != nil && (tranSecs = split.timeIntervalSince1970) < endSecs) {
                     nextDate = split;
                     nextSecs = tranSecs;
                 } else {
@@ -301,7 +302,7 @@ QTimeZonePrivate::Data QMacTimeZonePrivate::previousTransition(qint64 beforeMSec
     while (nextDate != nil && nextSecs < endSecs) {
         prevSecs = nextSecs;
         nextDate = [m_nstz nextDaylightSavingTimeTransitionAfterDate:nextDate];
-        nextSecs = [nextDate timeIntervalSince1970];
+        nextSecs = nextDate.timeIntervalSince1970;
         if (nextSecs <= prevSecs) // presumably no later data available
             break;
     }
@@ -316,18 +317,19 @@ QByteArray QMacTimeZonePrivate::systemTimeZoneId() const
 {
     // Reset the cached system tz then return the name
     [NSTimeZone resetSystemTimeZone];
-    return QString::fromNSString([[NSTimeZone systemTimeZone] name]).toUtf8();
+    Q_ASSERT(NSTimeZone.systemTimeZone);
+    return QString::fromNSString(NSTimeZone.systemTimeZone.name).toUtf8();
 }
 
 QList<QByteArray> QMacTimeZonePrivate::availableTimeZoneIds() const
 {
-    NSEnumerator *enumerator = [[NSTimeZone knownTimeZoneNames] objectEnumerator];
-    QByteArray tzid = QString::fromNSString([enumerator nextObject]).toUtf8();
+    NSEnumerator *enumerator = NSTimeZone.knownTimeZoneNames.objectEnumerator;
+    QByteArray tzid = QString::fromNSString(enumerator.nextObject).toUtf8();
 
     QList<QByteArray> list;
     while (!tzid.isEmpty()) {
         list << tzid;
-        tzid = QString::fromNSString([enumerator nextObject]).toUtf8();
+        tzid = QString::fromNSString(enumerator.nextObject).toUtf8();
     }
 
     std::sort(list.begin(), list.end());