Skip to content

Remove more superfluous semicolons after function definition#5521

Merged
ttorble merged 2 commits into
MicrosoftDocs:mainfrom
Rageking8:remove-more-superfluous-semicolons-after-function-definition
Jul 28, 2025
Merged

Remove more superfluous semicolons after function definition#5521
ttorble merged 2 commits into
MicrosoftDocs:mainfrom
Rageking8:remove-more-superfluous-semicolons-after-function-definition

Conversation

@Rageking8

Copy link
Copy Markdown
Contributor

Found more instances after #5393.

@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 4cea282:

✅ Validation status: passed

File Status Preview URL Details
docs/atl/codesnippet/CPP/atl-collection-classes_1.cpp ✅Succeeded
docs/c-runtime-library/reference/get-purecall-handler-set-purecall-handler.md ✅Succeeded
docs/code-quality/c28208.md ✅Succeeded
docs/cpp/class-templates.md ✅Succeeded
docs/cpp/decltype-cpp.md ✅Succeeded
docs/cpp/explicit-specialization-of-function-templates.md ✅Succeeded
docs/cpp/explicitly-defaulted-and-deleted-functions.md ✅Succeeded
docs/cpp/initializers.md ✅Succeeded
docs/cpp/safebuffers.md ✅Succeeded
docs/cpp/selectany.md ✅Succeeded
docs/cpp/single-inheritance.md ✅Succeeded
docs/cppcx/platform-object-class.md ✅Succeeded
docs/dotnet/double-thunking-cpp.md ✅Succeeded
docs/error-messages/compiler-errors-1/compiler-error-c2246.md ✅Succeeded
docs/error-messages/compiler-errors-1/compiler-error-c2249.md ✅Succeeded
docs/error-messages/compiler-errors-1/compiler-error-c2254.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c2614.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c2664.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c2724.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c2758.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c2765.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c3068.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c3771.md ✅Succeeded
docs/error-messages/compiler-warnings/compiler-warning-c4355.md ✅Succeeded
docs/error-messages/compiler-warnings/compiler-warning-level-1-c4183.md ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

@prmerger-automator

Copy link
Copy Markdown
Contributor

PRMerger Results

Issue Description
Changed Files This PR contains more than 10 changed files.
File Change Percent This PR contains file(s) with more than 30% file change.
Unapproved file extension(s) This PR contains file types that are not supported in the repo. The only supported file types are jpg, gif, png, json, svg and md.

@ShannonLeavitt

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 Jul 10, 2025
@v-dirichards

Copy link
Copy Markdown
Contributor

@TylerMSFT Could you review this proposed update to your article and enter #sign-off in a comment if it's ready to merge?

Thanks!

@TylerMSFT 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 removing the barnacles.

@TylerMSFT

Copy link
Copy Markdown
Collaborator

#sign-off

@ttorble ttorble merged commit fe193da into MicrosoftDocs:main Jul 28, 2025
2 checks passed
@Rageking8 Rageking8 deleted the remove-more-superfluous-semicolons-after-function-definition branch July 28, 2025 14:44
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