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

Impact of C++20 integer comparison functions on our code

    XMLWordPrintable

Details

    • Epic
    • Status: Open
    • P2: Important
    • Resolution: Unresolved
    • None
    • None
    • Core: Other
    • None
    • C++20 int compare @ Qt

    Description

      C++20 added integer comparison functions to the <utility> header (https://en.cppreference.com/w/cpp/utility#Integer_comparison_functions). While these don't have impact on our APIs, we may want to reimplement them under the q20 mechanism and start using them instead of the traditional ==/!=/</>/<=/>= for mixed-integer types, because developers all too often get these wrong.

      We might need to do something for qfloat16 and the q_le types.

      Attachments

        Issue Links

          is required for

          Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. QTBUG-99243 As a user of Qt, I'd like it to fully support C++20

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

          User Story - Created by Jira Software - do not edit or delete. Issue type for a user story. QTBUG-109360 Use C++20 code with Qt (Phase I)

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

          Suggestion - Public suggestions QTBUG-100881 qAddOverflow et al are susceptible to misuse (explicit template arguments), leading to False Negatives

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Reported
          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:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes