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

As a QLatin1String user, I'm puzzled that this class is a view

    XMLWordPrintable

    Details

    • Type: User Story
    • Status: Closed
    • Priority: P3: Somewhat important
    • Resolution: Fixed
    • Affects Version/s: 6.0, 6.1, 6.2
    • Fix Version/s: 6.4.0 Beta1
    • Labels:
      None
    • Sprint:
      Team One Foundation Sprint 52, Team A Foundation Sprint 53, Team A Foundation Sprint 54, Team A Foundation Sprint 55, Foundation Next
    • Commits:
      e440fec7fc (qt/qtbase/dev) e440fec7fc (qt/tqtc-qtbase/dev)

      Description

      I must confess that I'm new to Qt and still have a lot to learn. I already like the framework and value the API consistency. Similar stuff works similar and is similarly named. However, I always stumble over the issue that QLatin1String is a view like Q(Utf8)StringView, not an owning container, as I expected. I expected the view to be called QLatin1String_View_.

      Can you please fix this inconsistency?

      But instead of porting everything like s/QLatin1String/QLatin1StringView/, please provide

      namespace Qt {
      inline namespace string_literals {
       operator""_L1
      }
      }
      

      (something I use in every project and have to define by myself), but for string literals (→QLatin1StringView) and char (→QLatin1Char).

      Once you have that, I don't care what the things those operators return are named, as I won't see it in my code anymore (only in compiler error messages, but it's been years since I saw one of those ;P). I digress.

      Thanks!

        Attachments

          Issue Links

          For Gerrit Dashboard: QTBUG-98434
          # Subject Branch Project Status CR V

            Activity

              People

              Assignee:
              sonakur Sona Kurazyan
              Reporter:
              mmutz Marc Mutz
              PM Owner:
              Vladimir Minenko Vladimir Minenko
              RnD Owner:
              Alex Blasche Alex Blasche
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Gerrit Reviews

                  There is 1 open Gerrit change