Skip to content

Add usage assertions when deleting a queue#1449

Open
kstribrnAmzn wants to merge 2 commits into
FreeRTOS:mainfrom
kstribrnAmzn:queueDeleteUseAssertion
Open

Add usage assertions when deleting a queue#1449
kstribrnAmzn wants to merge 2 commits into
FreeRTOS:mainfrom
kstribrnAmzn:queueDeleteUseAssertion

Conversation

@kstribrnAmzn

Copy link
Copy Markdown
Member

Description

Adds an assertion which helps to verify
a queue is not in use before deletion.

Test Steps

Verified with a simple C program which directly called the queue with the list not-empty.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.
    • Not adding a UT for a simple assertion

Related Issue

Requested by user

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Adds an assertion which helps to verify
a queue is not in use before deletion.
@sonarqubecloud

Copy link
Copy Markdown

@kstribrnAmzn

Copy link
Copy Markdown
Member Author

Failing unit tests will be fixed in an upcoming FreeRTOS/FreeRTOS PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants