Skip to content

CRT_ASSERT is not a valid report type. Should be _CRT_ASSERT #4496

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 7, 2023
Merged

CRT_ASSERT is not a valid report type. Should be _CRT_ASSERT #4496

merged 1 commit into from
Apr 7, 2023

Conversation

LuShuchen
Copy link
Contributor

No description provided.

@prmerger-automator
Copy link
Contributor

@LuShuchen : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 0c0b6fe:

✅ Validation status: passed

File Status Preview URL Details
docs/c-runtime-library/reference/assert-asserte-assert-expr-macros.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Jak-MS
Copy link
Contributor

Jak-MS commented Apr 7, 2023

@TylerMSFT

  • Can you review this PR?
  • IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#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 Apr 7, 2023
Copy link
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

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

Thank you, @LuShuchen , for improving the docs!

@TylerMSFT
Copy link
Collaborator

#sign-off

@TylerMSFT TylerMSFT merged commit c31622d into MicrosoftDocs:main Apr 7, 2023
@LuShuchen LuShuchen deleted the patch-1 branch April 10, 2023 02:50
learn-build-service-prod bot added a commit that referenced this pull request Apr 13, 2023
* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

* Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main) (#4867)

* 3/31 AM Publish (#4859)

* fix SAL annotation

* acrolinx

* go with _Out_opt_ and annotate the sample with why

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

* Update assert-asserte-assert-expr-macros.md (#4496)

* 4/10/2023 AM Publish (#4868)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* Update attributes.md (#4498)

* 4/11/2023 AM Publish (#4869)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* remove unneeded `To`

* 4/12/2023 AM Publish (#4872)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

---------

Co-authored-by: huypub <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Taojunshen <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: huypub <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>
learn-build-service-prod bot added a commit that referenced this pull request Apr 14, 2023
* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

* Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main) (#4867)

* 3/31 AM Publish (#4859)

* fix SAL annotation

* acrolinx

* go with _Out_opt_ and annotate the sample with why

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

* Update assert-asserte-assert-expr-macros.md (#4496)

* 4/10/2023 AM Publish (#4868)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* Update attributes.md (#4498)

* 4/11/2023 AM Publish (#4869)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* remove unneeded `To`

* 4/12/2023 AM Publish (#4872)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

---------

Co-authored-by: huypub <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Taojunshen <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>

* alphabetize the attributes

* acrolinx

* Correct typo

* Bullet nonsequential list

* Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main) (#4878)

* 3/31 AM Publish (#4859)

* fix SAL annotation

* acrolinx

* go with _Out_opt_ and annotate the sample with why

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

* Update assert-asserte-assert-expr-macros.md (#4496)

* 4/10/2023 AM Publish (#4868)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* Update attributes.md (#4498)

* 4/11/2023 AM Publish (#4869)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* remove unneeded `To`

* 4/12/2023 AM Publish (#4872)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* 4/13/2023 AM Publish (#4876)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

* Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main) (#4867)

* 3/31 AM Publish (#4859)

* fix SAL annotation

* acrolinx

* go with _Out_opt_ and annotate the sample with why

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

* Update assert-asserte-assert-expr-macros.md (#4496)

* 4/10/2023 AM Publish (#4868)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* Update attributes.md (#4498)

* 4/11/2023 AM Publish (#4869)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* remove unneeded `To`

* 4/12/2023 AM Publish (#4872)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

---------

Co-authored-by: huypub <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Taojunshen <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: huypub <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>

* Remove unmatched left bracket in stl-containers.md

* Add missing quotation mark in algorithms.md

* Fix punctuation typo in functional.md

---------

Co-authored-by: huypub <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Taojunshen <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>
Co-authored-by: Martin Schonger <[email protected]>
Co-authored-by: Courtney Wales <[email protected]>

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: huypub <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>
Co-authored-by: Dennis Rea <[email protected]>
Co-authored-by: Martin Schonger <[email protected]>
Co-authored-by: Courtney Wales <[email protected]>
learn-build-service-prod bot added a commit that referenced this pull request Apr 17, 2023
…//github.com/MicrosoftDocs/cpp-docs (branch main) (#4867)

* 3/31 AM Publish (#4859)

* fix SAL annotation

* acrolinx

* go with _Out_opt_ and annotate the sample with why

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

* Update assert-asserte-assert-expr-macros.md (#4496)

* 4/10/2023 AM Publish (#4868)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* Update attributes.md (#4498)

* 4/11/2023 AM Publish (#4869)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* remove unneeded `To`

* 4/12/2023 AM Publish (#4872)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

---------

Co-authored-by: huypub <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Taojunshen <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>
learn-build-service-prod bot added a commit that referenced this pull request Apr 17, 2023
…//github.com/MicrosoftDocs/cpp-docs (branch main) (#4878)

* 3/31 AM Publish (#4859)

* fix SAL annotation

* acrolinx

* go with _Out_opt_ and annotate the sample with why

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

* Update assert-asserte-assert-expr-macros.md (#4496)

* 4/10/2023 AM Publish (#4868)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* Update attributes.md (#4498)

* 4/11/2023 AM Publish (#4869)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* remove unneeded `To`

* 4/12/2023 AM Publish (#4872)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* 4/13/2023 AM Publish (#4876)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

* Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main) (#4867)

* 3/31 AM Publish (#4859)

* fix SAL annotation

* acrolinx

* go with _Out_opt_ and annotate the sample with why

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

* Update assert-asserte-assert-expr-macros.md (#4496)

* 4/10/2023 AM Publish (#4868)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* Update attributes.md (#4498)

* 4/11/2023 AM Publish (#4869)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

* remove unneeded `To`

* 4/12/2023 AM Publish (#4872)

* alt-text updates

* acrolinx

* fix image links, acrolinx

* update alt-text

* fixup text

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>

---------

Co-authored-by: huypub <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Taojunshen <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: huypub <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>

* Remove unmatched left bracket in stl-containers.md

* Add missing quotation mark in algorithms.md

* Fix punctuation typo in functional.md

---------

Co-authored-by: huypub <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: LuShuchen <[email protected]>
Co-authored-by: Taojunshen <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Terry Mahaffey <[email protected]>
Co-authored-by: Igor Zhukov <[email protected]>
Co-authored-by: Shannon Leavitt <[email protected]>
Co-authored-by: Martin Schonger <[email protected]>
Co-authored-by: Courtney Wales <[email protected]>
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