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

QStringList TLC

    XMLWordPrintable

Details

    • QStringList-6.7
    • e1dcc858b (dev), e1fe8963a (6.6), ea5035bfb (tqtc/lts-6.5), 0d3585447 (tqtc/lts-6.2), ae62bd02f (tqtc/lts-5.15)

    Description

      QStringList has several shortcomings. This epic sketches them. When grooming, please create proper tasks for each item under this epic:

      In no particular order:

      • the docs show only a subset of functions
        • e.g. indexOf only for QRegularExpression, not for QString, QStringView, QLatin1StringView (which partly come in via the indexOf template function, but it looks odd nonetheless)
        • doc fixes should be backported to current LTS, at least, so may need a Q_QDOC-block to fake them before actually implementing them (which is for dev only, because it adds new symbols)
      • the following functions are missing (some) overloads (join and contains are considered complete, for now):
        • indexOf/lastIndexOf
          • they work because they have a template backend, but fact this is visible in Clang -ftime-trace (→ QTBUG-97601), so they should get the same treatment as contains (out-of-line QStringList_xxx functions
        • filter() (QL1SV)
      • replaceInStrings() should be QAnyStringView-ified (two arguments = too many overloads)
      • the following functions could do with a CaseSensitivity argument:
        • indexOf/lastIndexOf (cf. contains())
          • must take care where to place it w.r.t. the from parameter
        • removeDuplicates()?

      Attachments

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

        Activity

          People

            cnn Qt Core & Network
            mmutz Marc Mutz
            Vladimir Minenko Vladimir Minenko
            Alex Blasche Alex Blasche
            Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:

              Gerrit Reviews

                There are no open Gerrit changes