Skip to content

Conversation

philnik777
Copy link
Contributor

include_next doesn't work very well with the C++03 headers and modules. Since these specific headers are very self-contained there isn't much of a reason to split them into C++03/non-C++03 headers, so let's just remove them. The few C wrapper headers that aren't as self-contained will be refactored in a separate patch.

@philnik777 philnik777 force-pushed the cxx03_remove_c_wrapper_headers branch from 39ad356 to 5993f51 Compare October 16, 2025 13:26
Copy link

⚠️ C/C++ code formatter, clang-format found issues in your code. ⚠️

You can test this locally with the following command:
git-clang-format --diff origin/main HEAD --extensions ,h -- libcxx/include/__cxx03/__thread/support/pthread.h libcxx/include/__cxx03/cctype libcxx/include/__cxx03/cerrno libcxx/include/__cxx03/cfenv libcxx/include/__cxx03/cfloat libcxx/include/__cxx03/cinttypes libcxx/include/__cxx03/cstddef libcxx/include/__cxx03/cstdio libcxx/include/__cxx03/cwctype libcxx/include/__cxx03/ext/__hash libcxx/include/__cxx03/wchar.h libcxx/include/complex.h libcxx/include/ctype.h libcxx/include/errno.h libcxx/include/fenv.h libcxx/include/float.h libcxx/include/inttypes.h libcxx/include/stdbool.h libcxx/include/stddef.h libcxx/include/stdio.h libcxx/include/tgmath.h libcxx/include/wctype.h --diff_from_common_commit

⚠️
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing origin/main to the base branch/commit you want to compare against.
⚠️

View the diff from clang-format here.
diff --git a/libcxx/include/__cxx03/ext/__hash b/libcxx/include/__cxx03/ext/__hash
index e568b4a2b..e2c521568 100644
--- a/libcxx/include/__cxx03/ext/__hash
+++ b/libcxx/include/__cxx03/ext/__hash
@@ -14,8 +14,8 @@
 
 #include <__cxx03/__config>
 #include <__cxx03/cstring>
-#include <stddef.h>
 #include <__cxx03/string>
+#include <stddef.h>
 
 namespace __gnu_cxx {
 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant