On 1/24/12 9:47 AM, "ext John Layt" <[email protected]> wrote: > >On 24 Jan 2012 00:23, "John Layt" <[email protected]> wrote: >> >> On Monday 23 Jan 2012 22:48:12 [email protected] wrote: >> >> > I am very tempted to simply break this. localized binary numbers >>simply >> > don't make any sense. >> > >> > Cheers, >> > Lars >> >> Agreed it makes no sense. >> >> My initial thought was just to move the code to say QString/QByteArray >>and >> make it hard-coded C locale, i.e. not using QLocale, until I found that >> QLocale needs it too. If we remove the API from QLocale then this >>would be an >> option, just QTextStreamPrivate::putNumber() would need a little >>thought. >> >> John. >Note also that in QLocale it is only the toInt() etc methods that provide >the base option, the toString() methods do not. In the other classes it >goes both ways.
Ok. My vote goes towards simply removing the base from QLocale::toInt() and documenting that in the changes file. QTextStream should always stream in the "C" locale if the base is not decimal. Cheers, Lars _______________________________________________ Development mailing list [email protected] http://lists.qt-project.org/mailman/listinfo/development
