Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-81059

everything now includes <boost/preprocessor/control/iif.hpp>

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor - P4 Minor - P4
    • 7.2.0-rc0
    • None
    • None
    • None
    • Server Development Platform
    • Fully Compatible
    • Service Arch 2023-10-02

    Description

      An include for a rather obscure boost preprocessor header has been added by the IWYU sweep, to over 1500 c++ source files.

      #include <boost/preprocessor/control/iif.hpp>
      

      I don't really understand why IWYU would do this to a file like
      "src/mongo/db/commands/server_status_metric.h", which doesn't have any macros inside it at all. It seems like IWYU made a mistake here and it should be undone.

      I think maybe one of the other headers needs something from the iif.hpp header and IWYU didn't resolve that dependency appropriately.

      Attachments

        Activity

          People

            billy.donahue@mongodb.com Billy Donahue
            billy.donahue@mongodb.com Billy Donahue
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: