Skip to content

Conversation

Rageking8
Copy link
Contributor

  • Augment remarks with what happens without /Za flag
  • Tweak and format examples
    • Rename func to func1 for alignment
    • Tighten wording in comments
    • Remove superfluous spaces and add newline
  • Add each other as "See also" link since they are analogs
  • Update ms.date metadata

Copy link
Contributor

Learn Build status updates of commit 34f705f:

✅ Validation status: passed

File Status Preview URL Details
docs/error-messages/compiler-errors-1/compiler-error-c2190.md ✅Succeeded
docs/error-messages/compiler-errors-1/compiler-error-c2191.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-dirichards
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 22, 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

Updates documentation for compiler errors C2190 and C2191 to provide clearer explanations and better cross-references. The changes enhance the remarks section with behavior information when the /Za flag is not used, improve code examples with better naming and formatting, and add cross-references between the related error messages.

  • Enhanced remarks to explain compiler behavior without the /Za flag
  • Improved code examples with consistent naming and clearer comments
  • Added cross-references between C2190 and C2191 since they are related errors

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/error-messages/compiler-errors-1/compiler-error-c2191.md Updated remarks, improved example formatting, and added cross-reference to C2190
docs/error-messages/compiler-errors-1/compiler-error-c2190.md Updated remarks, improved example formatting, and added cross-reference to C2191

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