[Development] QTimeZone (was: Re: Qt 5.2 header diff: QtCore)

John Layt jlayt at kde.org
Tue Nov 5 14:32:03 CET 2013


On 5 November 2013 12:03, Marc Mutz <marc.mutz at kdab.com> wrote:
> On Tuesday, November 05, 2013 01:07:32 Thiago Macieira wrote:
>> +    // ### Qt 6: Merge with above with default offsetSeconds = 0
>> +    QDateTime(const QDate &date, const QTime &time, Qt::TimeSpec spec, int
>> offsetSeconds);
>> +#ifndef QT_BOOTSTRAPPED
>> +    QDateTime(const QDate &date, const QTime &time, const QTimeZone
>> &timeZone);
>> +#endif // QT_BOOTSTRAPPED
>
> IMO, it would be more readable to use QT_NO_TIMEZONE, which is then set by
> QT_BOOTSTRAPPED.

Yes, making time zones optional was something I was going to look at
for 5.3 along with configure flags to choose native or TZ/ICU time
zones.  However I could probably change just this if people want.
Someone would just need to point me to where/how to do the configure
magic :-)

John.



More information about the Development mailing list