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

[spike] Create a policy for [[nodiscard]] on ctors

    XMLWordPrintable

Details

    • Task
    • Resolution: Fixed
    • P2: Important
    • 6.6
    • None
    • Core: Other
    • None
    • 5
    • ca86afb97 (master)
    • Foundation Sprint 84, Foundation Sprint 85

    Description

      QTBUG-104161 will give us the go to apply [[nodiscard]] (or a wrapper macro) to constructors. This ticket is about finding out on which kinds of ctors we'd want this attribute, and getting buy-in for that from the community (dev ML?).

      Seeing the proliferation of [[nodiscard]] on almost all non-void-return functions, I wouldn't be surprised if the outcome is "all ctors" (at least the non-SMF¹ ones).

      ¹ SMF == Special Member Function

      Attachments

        Issue Links

          blocks

          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

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

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

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

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

          Task - A task that needs to be done. QTBUG-104161 [spike] Find out whether all supported compilers support [[nodiscard]] on ctor, add macro if not

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

          Task - A task that needs to be done. QTBUG-114767 Add Q_NODISCARD_X and Q_NODISCARD_CTOR_X, wrapping [[nodiscard("reason")]]

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          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