Skip to content

Conversation

Rageking8
Copy link
Contributor

Update error message, augment remarks, and add example.

Copy link
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.

Copy link
Contributor

PRMerger Results

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

@v-regandowner
Copy link
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
Contributor

@Copilot 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
Preview

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.

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.

4 participants