Skip to content
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

Add projects for out-of-proc WinRT API and test application #208

Merged
merged 9 commits into from
Apr 6, 2023

Conversation

abeltrano
Copy link
Contributor

Type

  • Bug fix
  • Feature addition
  • Feature update
  • Breaking change
  • Non-functional change
  • Documentation
  • Infrastructure

Goals

Put the project structure in place to enable full-fledged service API development.

Technical Details

  • Move VS solution, .gitgnore, and CMakeLists.txt to parent directory and rename appropriately.
  • Add a WinRT component project to contain the eventual out-of-proc C++/WinRT APIs.
  • Add a WinRT executable project to contain a service-like runtime that hosts the out-of-proc components.
    • This executable is meant to exercise registration-free WinRT components as they would be used in the service.

Test Results

Compile-tested only for now. Additional testing will be done in follow-up PRs when the future work is completed.

Reviewer Focus

None

Future Work

  • The test application needs to be tested and extended such that it doesn't exit immediately.
  • The in-proc APIs need to be modified to contain a private instance of the out-of-proc NearObjectService component.

Checklist

  • Build target all compiles cleanly.
  • clang-format and clang-tidy deltas produced no new output.
  • Newly added functions include doxygen-style comment block.

@abeltrano abeltrano requested a review from a team as a code owner April 6, 2023 02:48
@abeltrano abeltrano merged commit 46cbc5c into develop Apr 6, 2023
@abeltrano abeltrano deleted the winrtnoproject branch April 6, 2023 17:13
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.

2 participants