Automate enforcement of inclusion of mongo/platform/basic.h

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Duplicate
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Internal Code
    • Dev Platform 2021-08-09
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      All cpp files under src/mongo are technically required to include mongo/platform/basic.h as their first include. Over 400 files do not. While this does not lead to an ill effect today, we should either enforce the requirement or remove it.

      The requirement was originally put in place because Windows header inclusion is messy, and order often matters, so using the inclusion of basic.h to ensure proper windows header inclusion was useful. I do not know if it remains relevant.

            Assignee:
            Andrew Morrow (Inactive)
            Reporter:
            Andy Schwerin
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: