Skip to content

Add example and tweak C3187 error reference - #5665

Merged
Tracey Torble (ttorble) merged 7 commits into
MicrosoftDocs:mainfrom
Rageking8:add-example-and-tweak-c3187-error-reference
Feb 9, 2026
Merged

Tracey Torble (ttorble) merged 7 commits into
MicrosoftDocs:mainfrom
Rageking8:add-example-and-tweak-c3187-error-reference

Conversation

@Rageking8

Copy link
Copy Markdown
Contributor
  • Update error message (remove space before colon and replace "identifier" with "__func__")
  • Add __func__ link and tweak remarks (use "identifier" instead of "macro" as per docs/cpp/func.md)
  • Add new example showing 2 possible ways to emit C3187
  • Add similar C2457 link under "See also"
  • Add "Remarks" heading and update metadata

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit b8cdb92:

✅ Validation status: passed

File Status Preview URL Details
docs/error-messages/compiler-errors-2/compiler-error-c3187.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-errors-c3100-through-c3199.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-dirichards

Copy link
Copy Markdown
Contributor

Tyler Whitney (@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 15, 2025

@TylerMSFT Tyler Whitney (TylerMSFT) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for the example. I added a correct version to show how to fix it, which may be overly pedantic but it's done now.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 14c3fb4:

✅ Validation status: passed

File Status Preview URL Details
docs/error-messages/compiler-errors-2/compiler-error-c3187.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-errors-c3100-through-c3199.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 Tyler Whitney (TylerMSFT) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These go over well (showing how to correct the error). Thank you.

@TylerMSFT

Copy link
Copy Markdown
Collaborator

#sign-off

@ttorble
Tracey Torble (ttorble) merged commit 9765296 into MicrosoftDocs:main Feb 9, 2026
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.

5 participants