Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-25055

Qt 5 to-do's in QLocale

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: P2: Important
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: 5.0.0 Beta 1
    • Component/s: Core: Locales (i18n)
    • Labels:
      None
    • Technical Risk:
      Normal

      Description

      The QLocale class declaration in qlocale.h contains two Qt 5 to-do comments:

      qlocale.h
          // ### Qt 5: We need to return QString from these function since
          //           unicode data contains several characters for these fields.
          QChar decimalPoint() const;
          QChar groupSeparator() const;
          QChar percent() const;
          QChar zeroDigit() const;
          QChar negativeSign() const;
          QChar positiveSign() const;
          QChar exponential() const;
      ...
      private:
          friend struct QLocalePrivate;
          // ### We now use this field to pack an index into locale_data and NumberOptions.
          // ### Qt 5: change to a QLocaleData *d; uint numberOptions.
          union {
              void *v;
              Data p;
          };
      

      If these can be resolved without affecting source-compatibility, this should be deone for Qt 5.0.0. If not, the comments should be changed to "Qt 6" or removed.

        Attachments

          Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            Activity

              People

              Assignee:
              thiago Thiago Macieira
              Reporter:
              jasmcdon Jason McDonald (Closed Nokia Identity. Please assign to "macadder" instead) (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Gerrit Reviews

                  There are no open Gerrit changes