Skip to content

CDRIVER-6085 Reorder + annotate include directives with "IWYU pragma: export" #2167

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Nov 13, 2025
Merged

CDRIVER-6085 Reorder + annotate include directives with "IWYU pragma: export" #2167

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
739d320
Reorder and annotate component include directives with "IWYU pragma: …
eramongodb Oct 30, 2025
c46a1e2
Merge remote-tracking branch 'upstream/master' into HEAD
eramongodb Nov 13, 2025
7832c51
Apply "keep" pragma to unused headers in compat.h
eramongodb Nov 13, 2025
545b141
Component header for bson-private.h is bson_t.h
eramongodb Nov 13, 2025
0e747db
Rename bson-private.h -> bson_t-private.h
eramongodb Nov 13, 2025
abadc65
Drive-by Python script format fixes
eramongodb Nov 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .evergreen/config_generator/components/earthly.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ def earthly_task(

CONTAINER_RUN_DISTROS = [
'amazon2',
"debian11-latest-large",
'debian11-latest-large',
'debian12-latest-large',
'ubuntu2204-large',
'ubuntu2404-large',
Expand Down
6 changes: 3 additions & 3 deletions .evergreen/config_generator/components/oidc.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
from shrub.v3.evg_build_variant import BuildVariant
from shrub.v3.evg_command import EvgCommandType, ec2_assume_role, KeyValueParam, expansions_update
from shrub.v3.evg_command import EvgCommandType, KeyValueParam, ec2_assume_role, expansions_update
from shrub.v3.evg_task import EvgTask, EvgTaskRef
from shrub.v3.evg_task_group import EvgTaskGroup

from config_generator.components.funcs.run_tests import RunTests
from config_generator.components.funcs.fetch_det import FetchDET
from config_generator.components.funcs.fetch_source import FetchSource
from config_generator.components.funcs.run_tests import RunTests
from config_generator.components.sasl.openssl import SaslCyrusOpenSSLCompile
from config_generator.etc.utils import bash_exec
from config_generator.etc.distros import find_small_distro
from config_generator.etc.utils import bash_exec


def task_groups():
Expand Down
2 changes: 1 addition & 1 deletion .evergreen/legacy_config_generator/evergreen_config_lib/testazurekms.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ def _create_variant():
name='testazurekms-variant',
display_name='Azure KMS',
# Azure Virtual Machine created is Debian 11.
run_on='debian11-small', # TODO: switch to 'debian11-latest-small' after DEVPROD-23011 fixed.
run_on='debian11-small', # TODO: switch to 'debian11-latest-small' after DEVPROD-23011 fixed.
tasks=['testazurekms_task_group', 'testazurekms-fail-task'],
batchtime=20160,
) # Use a batchtime of 14 days as suggested by the CSFLE test README
Expand Down
10 changes: 5 additions & 5 deletions src/common/src/mlib/platform.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,15 +37,15 @@
#define _WIN32_WINNT 0x601
#endif
// Winsock must be included before windows.h
#include <winsock2.h>
#include <windows.h>
#include <winsock2.h> // IWYU pragma: export
#include <windows.h> // IWYU pragma: export
#endif

// POSIX headers
#if defined(__unix__) || defined(__unix) || defined(__APPLE__)
#include <unistd.h>
#include <fcntl.h>
#include <sys/types.h>
#include <unistd.h> // IWYU pragma: export
#include <fcntl.h> // IWYU pragma: export
#include <sys/types.h> // IWYU pragma: export
#endif

// clang-format on
Expand Down
5 changes: 3 additions & 2 deletions src/libbson/src/bson/bson-context-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,11 @@
#ifndef BSON_CONTEXT_PRIVATE_H
#define BSON_CONTEXT_PRIVATE_H

#include <bson/bson-context.h> // IWYU pragma: export

#include <common-thread-private.h>
//

#include <bson/bson-context.h>
#include <common-thread-private.h>


BSON_BEGIN_DECLS
Expand Down
5 changes: 4 additions & 1 deletion src/libbson/src/bson/bson-error-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,10 @@
#ifndef BSON_ERROR_PRIVATE_H
#define BSON_ERROR_PRIVATE_H

#include <bson/error.h>
#include <bson/error.h> // IWYU pragma: export

//

#include <bson/macros.h>


Expand Down
1 change: 1 addition & 0 deletions src/libbson/src/bson/bson-json-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
#ifndef BSON_JSON_PRIVATE_H
#define BSON_JSON_PRIVATE_H

#include <bson/bson-json.h> // IWYU pragma: export

struct _bson_json_opts_t {
bson_json_mode_t mode;
Expand Down
4 changes: 4 additions & 0 deletions src/libbson/src/bson/bson-vector-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,10 @@
#ifndef BSON_VECTOR_PRIVATE_H
#define BSON_VECTOR_PRIVATE_H

#include <bson/bson-vector.h> // IWYU pragma: export

//

#include <bson/compat.h>
#include <bson/macros.h>

Expand Down
2 changes: 2 additions & 0 deletions src/libbson/src/bson/bson-vector.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@
#define BSON_VECTOR_H

#include <bson/bson-endian.h>
#include <bson/bson-types.h>
#include <bson/bson_t.h>
#include <bson/compat.h>
#include <bson/macros.h>

Expand Down
2 changes: 1 addition & 1 deletion src/libbson/src/bson/bson-writer.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@

#include <bson/bson-writer.h>

#include <bson/bson-private.h>
#include <bson/bson_t-private.h>


struct _bson_writer_t {
Expand Down
2 changes: 1 addition & 1 deletion src/libbson/src/bson/bson.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@

#include <bson/bson-iso8601-private.h>
#include <bson/bson-json-private.h>
#include <bson/bson-private.h>
#include <bson/bson_t-private.h>
#include <bson/validate-private.h>
#include <common-json-private.h>
#include <common-macros-private.h>
Expand Down
44 changes: 23 additions & 21 deletions src/libbson/src/bson/bson.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,27 +20,29 @@

#define BSON_INSIDE

#include <bson/bson-bcon.h>
#include <bson/bson-clock.h>
#include <bson/bson-context.h>
#include <bson/bson-decimal128.h>
#include <bson/bson-iter.h>
#include <bson/bson-json.h>
#include <bson/bson-keys.h>
#include <bson/bson-oid.h>
#include <bson/bson-reader.h>
#include <bson/bson-string.h>
#include <bson/bson-types.h>
#include <bson/bson-utf8.h>
#include <bson/bson-value.h>
#include <bson/bson-vector.h>
#include <bson/bson-version-functions.h>
#include <bson/bson-writer.h>
#include <bson/compat.h>
#include <bson/config.h>
#include <bson/macros.h>
#include <bson/memory.h>
#include <bson/version.h>
#include <bson/bson-bcon.h> // IWYU pragma: export
#include <bson/bson-clock.h> // IWYU pragma: export
#include <bson/bson-context.h> // IWYU pragma: export
#include <bson/bson-decimal128.h> // IWYU pragma: export
#include <bson/bson-iter.h> // IWYU pragma: export
#include <bson/bson-json.h> // IWYU pragma: export
#include <bson/bson-keys.h> // IWYU pragma: export
#include <bson/bson-oid.h> // IWYU pragma: export
#include <bson/bson-reader.h> // IWYU pragma: export
#include <bson/bson-string.h> // IWYU pragma: export
#include <bson/bson-types.h> // IWYU pragma: export
#include <bson/bson-utf8.h> // IWYU pragma: export
#include <bson/bson-value.h> // IWYU pragma: export
#include <bson/bson-vector.h> // IWYU pragma: export
#include <bson/bson-version-functions.h> // IWYU pragma: export
#include <bson/bson-writer.h> // IWYU pragma: export
#include <bson/bson_t.h> // IWYU pragma: export
#include <bson/compat.h> // IWYU pragma: export
#include <bson/config.h> // IWYU pragma: export
#include <bson/error.h> // IWYU pragma: export
#include <bson/macros.h> // IWYU pragma: export
#include <bson/memory.h> // IWYU pragma: export
#include <bson/version.h> // IWYU pragma: export

#include <string.h>
#include <time.h>
Expand Down
3 changes: 3 additions & 0 deletions src/libbson/src/bson/bson-private.h → src/libbson/src/bson/bson_t-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@
#ifndef BSON_PRIVATE_H
#define BSON_PRIVATE_H

#include <bson/bson_t.h> // IWYU pragma: export

//

#include <bson/bson-types.h>
#include <bson/macros.h>
Expand Down
45 changes: 23 additions & 22 deletions src/libbson/src/bson/compat.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@
#endif
#endif

#include <bson/config.h>
#include <bson/macros.h>
#include <bson/config.h> // IWYU pragma: export
#include <bson/macros.h> // IWYU pragma: export


#ifdef BSON_OS_WIN32
Expand All @@ -42,46 +42,47 @@
#ifndef NOMINMAX
#define NOMINMAX
#endif
#include <winsock2.h>
#include <winsock2.h> // IWYU pragma: export
#ifndef WIN32_LEAN_AND_MEAN
#define WIN32_LEAN_AND_MEAN
#include <windows.h>
#include <windows.h> // IWYU pragma: export
#undef WIN32_LEAN_AND_MEAN
#else
#include <windows.h>
#include <windows.h> // IWYU pragma: export
#endif
#include <direct.h>
#include <io.h>
#include <direct.h> // IWYU pragma: export
#include <io.h> // IWYU pragma: export
#endif


#ifdef BSON_OS_UNIX
#include <sys/time.h>
#include <unistd.h>
#include <sys/time.h> // IWYU pragma: export
#include <sys/types.h> // IWYU pragma: export
#include <unistd.h> // IWYU pragma: export
#endif


#include <bson/macros.h>

#include <fcntl.h>
#include <sys/stat.h>
#include <fcntl.h> // IWYU pragma: export
#include <sys/stat.h> // IWYU pragma: export

#include <ctype.h>
#include <errno.h>
#include <limits.h>
#include <stdarg.h>
#include <stdbool.h>
#include <stdint.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
#include <ctype.h> // IWYU pragma: keep: to be removed.
#include <errno.h> // IWYU pragma: keep: to be removed.
#include <limits.h> // IWYU pragma: export
#include <stdarg.h> // IWYU pragma: export
Copy link
Collaborator

@kevinAlbs kevinAlbs Nov 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove unneeded export pragma? Later code references va_copy:

#if !defined(va_copy) && defined(__va_copy)

Copy link
Contributor Author

@eramongodb eramongodb Nov 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a compatibility header, this header conditionally provides va_copy when <stdarg.h> does not. Therefore, this header transitively provides <stdarg.h>.

#include <stdbool.h> // IWYU pragma: export
#include <stdint.h> // IWYU pragma: export
#include <stdio.h> // IWYU pragma: keep: to be removed.
#include <stdlib.h> // IWYU pragma: keep: to be removed.
#include <string.h> // IWYU pragma: keep: to be removed.
#include <time.h> // IWYU pragma: keep: to be removed.


BSON_BEGIN_DECLS

#if !defined(_MSC_VER) || (_MSC_VER >= 1800)
#include <inttypes.h>
#include <inttypes.h> // IWYU pragma: export
#endif
#ifdef _MSC_VER
#ifndef __cplusplus
Expand Down
2 changes: 1 addition & 1 deletion src/libbson/src/bson/macros.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
#ifndef BSON_MACROS_H
#define BSON_MACROS_H

#include <bson/config.h>
#include <bson/config.h> // IWYU pragma: export

#include <stdio.h>
#include <stdlib.h>
Expand Down
2 changes: 1 addition & 1 deletion src/libbson/tests/test-bson.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
*/


#include <bson/bson-private.h>
#include <bson/bson_t-private.h>
#include <bson/validate-private.h>
#include <common-bson-dsl-private.h>

Expand Down
4 changes: 3 additions & 1 deletion src/libmongoc/src/mongoc/mongoc-apm-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@
#ifndef MONGOC_APM_PRIVATE_H
#define MONGOC_APM_PRIVATE_H

#include <mongoc/mongoc-apm.h>
#include <mongoc/mongoc-apm.h> // IWYU pragma: export

//

#include <bson/bson.h>

Expand Down
4 changes: 4 additions & 0 deletions src/libmongoc/src/mongoc/mongoc-bulk-operation-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,10 @@
#ifndef MONGOC_BULK_OPERATION_PRIVATE_H
#define MONGOC_BULK_OPERATION_PRIVATE_H

#include <mongoc/mongoc-bulk-operation.h> // IWYU pragma: export

//

#include <mongoc/mongoc-array-private.h>
#include <mongoc/mongoc-write-command-private.h>

Expand Down
5 changes: 4 additions & 1 deletion src/libmongoc/src/mongoc/mongoc-change-stream-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,13 @@
#ifndef MONGOC_CHANGE_STREAM_PRIVATE_H
#define MONGOC_CHANGE_STREAM_PRIVATE_H

#include <mongoc/mongoc-change-stream.h> // IWYU pragma: export

//

#include <mongoc/mongoc-opts-helpers-private.h>
#include <mongoc/mongoc-opts-private.h>

#include <mongoc/mongoc-change-stream.h>
#include <mongoc/mongoc-client-session.h>
#include <mongoc/mongoc-collection.h>
#include <mongoc/mongoc-cursor.h>
Expand Down
5 changes: 4 additions & 1 deletion src/libmongoc/src/mongoc/mongoc-client-pool-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,12 @@
#ifndef MONGOC_CLIENT_POOL_PRIVATE_H
#define MONGOC_CLIENT_POOL_PRIVATE_H

#include <mongoc/mongoc-client-pool.h> // IWYU pragma: export

//

#include <mongoc/mongoc-topology-private.h>

#include <mongoc/mongoc-client-pool.h>
#include <mongoc/mongoc-topology-description.h>

#include <bson/bson.h>
Expand Down
5 changes: 4 additions & 1 deletion src/libmongoc/src/mongoc/mongoc-client-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,15 @@
#ifndef MONGOC_CLIENT_PRIVATE_H
#define MONGOC_CLIENT_PRIVATE_H

#include <mongoc/mongoc-client.h> // IWYU pragma: export

//

#include <mongoc/mongoc-apm-private.h>
#include <mongoc/mongoc-buffer-private.h>
#include <mongoc/mongoc-cluster-private.h>
#include <mongoc/mongoc-rpc-private.h>

#include <mongoc/mongoc-client.h>
#include <mongoc/mongoc-config.h>
#include <mongoc/mongoc-host-list.h>
#include <mongoc/mongoc-opcode.h>
Expand Down
4 changes: 3 additions & 1 deletion src/libmongoc/src/mongoc/mongoc-client-session-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@
#ifndef MONGOC_CLIENT_SESSION_PRIVATE_H
#define MONGOC_CLIENT_SESSION_PRIVATE_H

#include <mongoc/mongoc-client-session.h>
#include <mongoc/mongoc-client-session.h> // IWYU pragma: export

//

#include <bson/bson.h>

Expand Down
5 changes: 4 additions & 1 deletion src/libmongoc/src/mongoc/mongoc-client-side-encryption-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,14 @@
#ifndef MONGOC_CLIENT_SIDE_ENCRYPTION_PRIVATE_H
#define MONGOC_CLIENT_SIDE_ENCRYPTION_PRIVATE_H

#include <mongoc/mongoc-client-side-encryption.h> // IWYU pragma: export

//

#include <mongoc/mongoc-cmd-private.h>
#include <mongoc/mongoc-topology-private.h>

#include <mongoc/mongoc-client-pool.h>
#include <mongoc/mongoc-client-side-encryption.h>
#include <mongoc/mongoc-client.h>

#include <bson/bson.h>
Expand Down
4 changes: 4 additions & 0 deletions src/libmongoc/src/mongoc/mongoc-collection-private.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,10 @@
#ifndef MONGOC_COLLECTION_PRIVATE_H
#define MONGOC_COLLECTION_PRIVATE_H

#include <mongoc/mongoc-collection.h> // IWYU pragma: export

//

#include <mongoc/mongoc-client.h>

#include <bson/bson.h>
Expand Down
Loading