On Tuesday, November 05, 2013 14:32:03 John Layt wrote: > On 5 November 2013 12:03, Marc Mutz <[email protected]> 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 :-)
For booststrapping, just git-grep for QT_NO_QOBJECT. You also should add it to in the style you find there to src/corelib/global/qconfig-*.h, but that's entirely optional and I don't know the rules regarding changing those files. _______________________________________________ Development mailing list [email protected] http://lists.qt-project.org/mailman/listinfo/development
