[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