Skip to content

Add example for C2290 error reference#5701

Merged
ShannonLeavitt merged 5 commits into
MicrosoftDocs:mainfrom
Rageking8:add-example-for-c2290-error-reference
Dec 5, 2025
Merged

Add example for C2290 error reference#5701
ShannonLeavitt merged 5 commits into
MicrosoftDocs:mainfrom
Rageking8:add-example-for-c2290-error-reference

Conversation

@Rageking8
Copy link
Copy Markdown
Contributor

Update error message, augment remarks, and add example.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit b3e3f96:

✅ Validation status: passed

File Status Preview URL Details
docs/error-messages/compiler-errors-1/compiler-error-c2290.md ✅Succeeded

For more details, please refer to the build report.

@prmerger-automator
Copy link
Copy Markdown
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@v-regandowner
Copy link
Copy Markdown
Contributor

@TylerMSFT - Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

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

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged Tracking label for the PR review team label Aug 25, 2025
@v-dirichards v-dirichards self-assigned this Sep 24, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the documentation for compiler error C2290 by updating the error message format, enhancing the remarks section with helpful links, and adding a practical code example. The changes make the error reference more comprehensive and useful for developers encountering this inline assembly syntax issue.

  • Updated error message formatting to use proper quotation marks around 'asm'
  • Enhanced remarks section with links to relevant inline assembler documentation
  • Added a complete code example demonstrating both the error-causing syntax and the correct alternative

## Remarks

The **`asm`** syntax is reserved for future use.
The **`asm`** syntax is reserved for future use, try [`__asm`](../../assembler/inline/asm.md) instead. For more information, see [Inline Assembler](../../assembler/inline/inline-assembler.md).
Copy link

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider adding a space after the comma for better readability: 'reserved for future use. Try [__asm]...' instead of 'reserved for future use, try [__asm]...'

Copilot uses AI. Check for mistakes.
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 7d5b27e:

✅ Validation status: passed

File Status Preview URL Details
docs/error-messages/compiler-errors-1/compiler-error-c2290.md ✅Succeeded

For more details, please refer to the build report.

@prmerger-automator
Copy link
Copy Markdown
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@TylerMSFT
Copy link
Copy Markdown
Collaborator

#sign-off

@TylerMSFT
Copy link
Copy Markdown
Collaborator

#sign-off

@ShannonLeavitt ShannonLeavitt merged commit 6582476 into MicrosoftDocs:main Dec 5, 2025
2 checks passed
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.

6 participants