Skip to content

Remove superfluous semicolons after function definition#5393

Merged
v-regandowner merged 2 commits into
MicrosoftDocs:mainfrom
Rageking8:remove-superfluous-semicolons-after-function-definition
Jun 9, 2025
Merged

Remove superfluous semicolons after function definition#5393
v-regandowner merged 2 commits into
MicrosoftDocs:mainfrom
Rageking8:remove-superfluous-semicolons-after-function-definition

Conversation

@Rageking8

@Rageking8 Rageking8 commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Remove superfluous semicolons after function definition, with bunch of metadata edits.

@prmerger-automator prmerger-automator Bot requested a review from TylerMSFT June 6, 2025 10:31
@prmerger-automator

prmerger-automator Bot commented Jun 6, 2025

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

learn-build-service-prod Bot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 39135ed:

✅ Validation status: passed

File Status Preview URL Details
docs/cpp/function-overloading.md ✅Succeeded
docs/cpp/overload-resolution-of-function-template-calls.md ✅Succeeded
docs/dotnet/how-to-access-characters-in-a-system-string.md ✅Succeeded
docs/dotnet/how-to-define-and-use-delegates-cpp-cli.md ✅Succeeded
docs/error-messages/compiler-errors-1/compiler-error-c2134.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c2835.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c3185.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c3536.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c3672.md ✅Succeeded
docs/error-messages/compiler-warnings/compiler-warning-level-1-c4269.md ✅Succeeded
docs/error-messages/compiler-warnings/compiler-warning-level-3-c4191.md ✅Succeeded
docs/error-messages/compiler-warnings/compiler-warning-level-3-c4243.md ✅Succeeded
docs/error-messages/compiler-warnings/compiler-warning-level-4-c4205.md ✅Succeeded
docs/error-messages/compiler-warnings/compiler-warning-level-4-c4254.md ✅Succeeded
docs/error-messages/tool-errors/linker-tools-error-lnk2020.md ✅Succeeded
docs/error-messages/tool-errors/linker-tools-error-lnk2033.md ✅Succeeded
docs/extensions/how-to-declare-interior-pointers-with-the-const-keyword-cpp-cli.md ✅Succeeded
docs/extensions/how-to-overload-functions-with-interior-pointers-and-native-pointers-cpp-cli.md ✅Succeeded
docs/extensions/pin-ptr-cpp-cli.md ✅Succeeded
docs/extensions/string-cpp-component-extensions.md ✅Succeeded
docs/overview/cpp-conformance-improvements-2019.md ✅Succeeded
docs/standard-library/auto-ptr-class.md ✅Succeeded
docs/standard-library/money-get-class.md ✅Succeeded
docs/standard-library/moneypunct-class.md ✅Succeeded
docs/standard-library/numpunct-class.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

prmerger-automator Bot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

PRMerger Results

Issue Description
Changed Files This PR contains more than 10 changed files.

@v-regandowner

v-regandowner commented Jun 6, 2025

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 Jun 6, 2025
TylerMSFT
TylerMSFT approved these changes Jun 6, 2025
View reviewed changes

@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.

Those were curious. Thanks for the updates.

@TylerMSFT

TylerMSFT commented Jun 6, 2025

Copy link
Copy Markdown
Collaborator

#sign-off

@v-regandowner v-regandowner merged commit 9072050 into MicrosoftDocs:main Jun 9, 2025
2 checks passed
@Rageking8 Rageking8 deleted the remove-superfluous-semicolons-after-function-definition branch June 9, 2025 15:34
@Rageking8 Rageking8 mentioned this pull request Jun 12, 2025
Merged
@Rageking8 Rageking8 mentioned this pull request Jul 10, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TylerMSFT TylerMSFT TylerMSFT approved these changes

Assignees

@TylerMSFT TylerMSFT

Labels

aq-pr-triaged Tracking label for the PR review team Change sent to author needs-human-review ready-to-merge standard-libraries/subsvc visual-cpp/svc

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Rageking8 @v-regandowner @TylerMSFT