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

Add Q_NODISCARD_X and Q_NODISCARD_CTOR_X, wrapping [[nodiscard("reason")]]

    XMLWordPrintable

Details

    • Task
    • Resolution: Fixed
    • P2: Important
    • 6.7.0 FF
    • None
    • Core: Other
    • None
    • 3
    • 175423865 (dev), 38cd3cb12 (dev), 410b57380 (master)
    • Foundation Sprint 84, Foundation Sprint 85

    Description

      I have found a use of [[nodiscard("reason")]], so let's add the macro.

      Acceptance criteria:

      • add the macros
      • use them in at least one place
      • update QUIP-0019

      Attachments

        Issue Links

          relates to

          Task - A task that needs to be done. QTBUG-104167 [spike] Create a policy for [[nodiscard]] on ctors

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          resulted from

          Task - A task that needs to be done. QTBUG-104164 Apply [[nodiscard]] to ctors where useful [QtBase]

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          For Gerrit Dashboard: QTBUG-114767
          # Subject Branch Project Status CR V
          486792,7 [normative] Update QUIP-0019 for C++20 [[nodiscard("reason")]] master meta/quips Status: MERGED +2 0
          486798,6 Short live Q_NODISCARD_(CTOR_)X! dev qt/qtbase Status: MERGED +2 0
          486799,7 QFutureSynchonizer: suggest future.waitForFinished() instead of QFutureSynchronizer(future); dev qt/qtbase Status: MERGED +2 0

          Activity

            People

              mmutz Marc Mutz
              mmutz Marc Mutz
              Vladimir Minenko Vladimir Minenko
              Alex Blasche Alex Blasche
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes