Skip to content

Repo sync for protected CLA branch #4223

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

Merged
merged 19 commits into from
Oct 14, 2022
Merged

Conversation

opbld15
Copy link
Contributor

@opbld15 opbld15 commented Oct 14, 2022

The pull request is created from main638013656556537349sync_temp to main to fix git push error for protected CLA branch

colin-home and others added 19 commits October 10, 2022 17:46
Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main)
…llowup

Followup to cpp-docs 4202 to fix style, grammar issues
…//github.com/MicrosoftDocs/cpp-docs (branch main) (#4571)

* 10/11/2022 AM Publish (#4570)

* Document ARM64 unwind handling for return address signing

* Update date, fix step numbers

* Fix style, grammar in Arm64 Exception

Co-authored-by: opbld17 <[email protected]>
Co-authored-by: Colin Robertson <[email protected]>
Co-authored-by: opbld16 <[email protected]>
Co-authored-by: opbld15 <[email protected]>
Co-authored-by: Martin Storsjö <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

* example needed to mitigate dev floundering (#4210)

It is not easy to match up variant vt types with POD types. Especially since there is a gradual transition to modern C++. I used a number of dev hours on attempts to match the variant type, vt, to make the true/false work. The m_bValue type for the simple property was a BOOL  type, but trying to create a variant of type BOOL directly didn't work. Only by looking at VC Sample code for VisualStudioDemo did I see the (_variant_t)true construct, which becomes the standard_cast contruct in modern C++. I figured my Example code could make other programmer's efforts easier.

Co-authored-by: Taojunshen <[email protected]>
Co-authored-by: opbld17 <[email protected]>
Co-authored-by: Colin Robertson <[email protected]>
Co-authored-by: opbld16 <[email protected]>
Co-authored-by: opbld15 <[email protected]>
Co-authored-by: Martin Storsjö <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: rtischer8277 <[email protected]>
Resolve syncing conflicts from FromPublicMasterBranch to main
…//github.com/MicrosoftDocs/cpp-docs (branch main) (#4573)

* Use the correct canonical opcode name in the packed unwind table

This was a leftover from the original tentative name before the
real one was known.

* Correct Month range in `ok` member description (#4215)

Co-authored-by: Martin Storsjö <[email protected]>
Co-authored-by: Colin Robertson <[email protected]>
Co-authored-by: Christopher Doyle <[email protected]>
cleanup empty remarks and tidy up code example formatting
Adding COM dependency explanation for relog function
removing links to avoid pr issues
Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main)
Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main)
@opbld31
Copy link

opbld31 commented Oct 14, 2022

Learn Build status updates of commit 83d7da5:

✅ Validation status: passed

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
Commits Count This PR contains more than 15 commits.

@jborsecnik jborsecnik added the aq-pr-triaged Tracking label for the PR review team label Oct 14, 2022
@colin-home colin-home closed this Oct 14, 2022
@colin-home colin-home reopened this Oct 14, 2022
@opbld32
Copy link

opbld32 commented Oct 14, 2022

Learn Build status updates of commit 83d7da5:

✅ Validation status: passed

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
Commits Count This PR contains more than 15 commits.

@prmerger-automator
Copy link
Contributor

@huypub : Thanks for your contribution! The author(s) have been notified to review your proposed change.

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

Successfully merging this pull request may close these issues.

9 participants