Uploaded image for project: 'Qt Creator'
  1. Qt Creator
  2. QTCREATORBUG-12166

Use #pragma once in header templates.

    XMLWordPrintable

Details

    • aaa8beab88dddd7218f6d3c30fb29c04679e2098 (qt-creator/qt-creator/master)

    Description

      Auto-generated headers currently use #include guards. It is better to use #pragma once for several reasons, so I suggest the default is changed. The reasons are:

      1. It's less verbose and ugly.
      2. It's less error-prone - e.g. no risk of two header files using the same guard define.
      3. It's faster to compile.
      4. It's not as much of a dirty hack.
      5. It's been supported on all major compilers for years.

      I realise this may attract a degree of "We've always done it this way and it works fine. Why would you want to improve things?" stuck-in-the-muds, but I hope you will ignore them.

      Attachments

        Issue Links

          is duplicated by

          Suggestion - Public suggestions QTCREATORBUG-16816 Add an option to use #pragma once instead of the classical header guards when generating C/C++ headers

          • Not Evaluated - Not yet evaluated/prioritized
          • Closed
          relates to

          Suggestion - Public suggestions QTCREATORBUG-18679 Add quick fix to change header guard to #pragma once and vice versa

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Reported

          Suggestion - Public suggestions QTCREATORBUG-18110 Some settings are missing per-project configuration

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Open
          For Gerrit Dashboard: QTCREATORBUG-12166
          # Subject Branch Project Status CR V
          249123,10 Wizards: Support using #pragma once instead of include guards master qt-creator/qt-creator Status: MERGED -2 0

          Activity

            People

              fbucek fbucek
              timmmm Tim
              Votes:
              21 Vote for this issue
              Watchers:
              15 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes