Skip to content

Add __preserve_none documentation#5894

Closed
HulonJenkins wants to merge 4 commits intoMicrosoftDocs:mainfrom
HulonJenkins:dev/hulonjenkins/preserve_none
Closed

Add __preserve_none documentation#5894
HulonJenkins wants to merge 4 commits intoMicrosoftDocs:mainfrom
HulonJenkins:dev/hulonjenkins/preserve_none

Conversation

@HulonJenkins
Copy link
Contributor

No description provided.

@prmerger-automator
Copy link
Contributor

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

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 91416dd:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/cpp/preservenone.md ⚠️Warning Details
docs/cpp/toc.yml ⚠️Warning Details
docs/build/reference/decorated-names.md ✅Succeeded
docs/c-language/summary-of-declarations.md ✅Succeeded
docs/cpp/argument-passing-and-naming-conventions.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-errors-c7500-through-c7999.md ✅Succeeded
docs/error-messages/tool-errors/name-decoration.md ✅Succeeded

docs/cpp/preservenone.md

  • Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
  • Line 65, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'argument-passing-and-naming-conventions'.
  • Line 66, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../build/x64-calling-convention'.
  • Line 67, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'keywords-cpp'.
  • Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
  • Line 65, Column 1: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: argument-passing-and-naming-conventions?view=msvc-170
  • Line 66, Column 1: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: ../build/x64-calling-convention?view=msvc-170
  • Line 67, Column 1: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: keywords-cpp?view=msvc-170

docs/cpp/toc.yml

  • Line 648, Column 25: [Warning: file-not-found - See documentation] Invalid file link: '../cpp/preserve-none.md'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@@ -0,0 +1,67 @@
# __preserve_none
Copy link
Collaborator

Choose a reason for hiding this comment

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

The PR reviewers may ask you to change the filename to something like preserve-none.md Might do it now to save yourself the bother later.


## See also

[Argument Passing and Naming Conventions](argument-passing-and-naming-conventions?view=msvc-170)
Copy link
Collaborator

@TylerMSFT TylerMSFT Feb 11, 2026

Choose a reason for hiding this comment

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

Remove the ?view= stuff. Looks like a cut-n-paste error from an URL

| Compiler error C7740 | cannot jump to case label |
| Compiler error C7741 | ABI inconsistency: '*function*' was originally assumed to use '`C`' return semantics but now it requires '`C++`' return semantics |
| [Compiler error C7742](compiler-error-c7742.md) | '*identifier*': a forward declaration of an enum can only use a simple identifier |
| Compiler error C7743 | `__preserve_none` calling convention is not supported |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you link to the topic here.

Copy link
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

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

Handful of little things to fix.

@ttorble
Copy link
Contributor

ttorble commented Feb 11, 2026

@TylerMSFT @HulonJenkins

This content requires automated and human checks available only in the private repository. Please close this PR (#please-close) and move the commits to the private repository. If you need help moving the commits, contact the publicprs alias.

Note that words in file names must be separated with a hyphen (the TOC entry appears to be correct). The new article will also need metadata.

@HulonJenkins

For reference, Microsoft authors and contributors should make content contributions through the private repository so the content can be staged and validated by the current validation rules.

Read about authors working in the public repo for more information.

If you need help getting set up to work in the private repo, see Get started contributing to the Microsoft Learn platform.

Note that you need to sign in to Learn to access the articles.

label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@ttorble ttorble added the aq-pr-triaged Tracking label for the PR review team label Feb 11, 2026
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit f3077d8:

✅ Validation status: passed

File Status Preview URL Details
docs/build/reference/decorated-names.md ✅Succeeded
docs/c-language/summary-of-declarations.md ✅Succeeded
docs/cpp/argument-passing-and-naming-conventions.md ✅Succeeded
docs/cpp/preserve-none.md ✅Succeeded
docs/cpp/toc.yml ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-errors-c7500-through-c7999.md ✅Succeeded
docs/error-messages/tool-errors/name-decoration.md ✅Succeeded

For more details, please refer to the build report.

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.

@TylerMSFT
Copy link
Collaborator

We already moved this to the private repo. Neglected to close this, which I'll do now.

@TylerMSFT TylerMSFT closed this Feb 12, 2026
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.

3 participants