-
Notifications
You must be signed in to change notification settings - Fork 274
C++20 module support v1 (import winrt;) #1556
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.
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
Open
DefaultRyan
wants to merge
49
commits into
master
Choose a base branch
from
feature/modules_v1
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 5 commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
689582d
Initial rough crack at monolithic module consumption
DefaultRyan 9a29dd1
Try import std in tests. Requires PlatformToolset=v145
DefaultRyan 1652e6b
Exclude v145-specific project from batch builds.
DefaultRyan 79d8003
Attempt detecting ability to import std inside winrt
DefaultRyan 546491d
Test component project is building. Checkpoint with some documentatio…
DefaultRyan 01f5f03
PR feedback: split includes into std and non-std, add base_module_mac…
DefaultRyan 797fea7
Explicitly include <ratio> header in base_std_includes.h
DefaultRyan 5cddeb3
Add base_std_includes.h to natvis project
DefaultRyan da640ca
PR feedback: Reduce code duplication in base_macros.h, extern linkage…
DefaultRyan f9b0871
Add base_module_macros.h to natvis pch
DefaultRyan 21eb38c
Workaround access_token issue. More tests.
DefaultRyan 3ea319b
Revert dual-build of await_adapter.cpp
DefaultRyan 9c021d2
Fix some more test pch issues
DefaultRyan 2ea8a9a
Update module limitation docs
DefaultRyan afe92af
Export impl namespace to get component authoring to work.
DefaultRyan 8abb336
Enable VS2022 support!
DefaultRyan abae82c
For some reason, the project dependencies got clobbered
DefaultRyan 92ea106
import std in components should be conditional
DefaultRyan c78fc14
Enable building module test projects
DefaultRyan 22c67d8
Don't export impl from fast_forward
DefaultRyan 9250b45
Fix stuff around pch inclusion and component files. Updated docs.
DefaultRyan eb56247
Add some Copilot instructions for future sessions.
DefaultRyan e410940
Merge branch 'master' into feature/modules_v1
DefaultRyan d82e886
Remove stale ixx component folding code.
DefaultRyan 78c7344
Clean up some duplication in generated files.
DefaultRyan 4854d8f
Merge branch 'master' into feature/modules_v1
DefaultRyan 322a103
Add some #endif comments
DefaultRyan 5c8939c
Add tests to ensure cpp23 with import std work.
DefaultRyan c49874f
Merge branch 'master' into feature/modules_v1
DefaultRyan 851c745
Add nuget test projects. Replace cppwinrt -module with component #ifd…
DefaultRyan 86c210f
Test modules in CI
DefaultRyan 9ef6792
TestModuleComponent now has multiple namespaces to further check WINR…
DefaultRyan 4ff8c7b
Explicitly generate list of module namespaces and use that for header…
DefaultRyan e25fb9a
WINRT_IMPL_SKIP_INCLUDES is finally gone
DefaultRyan a6eb07e
Define missinf WINRT_MODULE for test_cpp23_module project
DefaultRyan b0f6665
Add module namespace filtering
DefaultRyan 272113b
Add module_filter to modules.instructions.md
DefaultRyan e87427c
(Checkpoint) inclusion with WINRT_MODULE works, but requires WINRT_MO…
DefaultRyan 7bc1c11
Full and clean inclusion via auto-import. Cleanup of terminology and …
DefaultRyan 7d2b8fe
Fir sln project dependencies and module project guids
DefaultRyan 3542a4c
Enforcing include guards only on the module boundary. It works.
DefaultRyan 31dd7db
Build cppwinrt23 test projects in solution
DefaultRyan 703490e
PR feedback items
DefaultRyan 2bd4342
Import-then-include works via extern "C++"
DefaultRyan f76820e
Revert accidentally added description file.
DefaultRyan ff34fd9
Merge branch 'master' into feature/modules_v1
DefaultRyan f6ecdb9
Cross-namespace dependencies in component authoring
DefaultRyan b83ea95
Merge branch 'feature/modules_v1' of https://github.com/microsoft/cpp…
DefaultRyan 9873877
Test cross-namespace base class dependency guarding
DefaultRyan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.