Skip to content

Rename variable ulStackDepth in ARMv8M ports #961

New issue

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

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

Already on GitHub? # to your account

Closed

Conversation

kar-rahul-aws
Copy link
Member

Description

This PR updates variable name ulStackDepth to uxStackDepth in defintion of function vPortStoreTaskMPUSettings as per the change introduced by PR# 942.
It also renames usStackDepth to uxStackDepth in some of the missing potable files in the aforementioned PR.

Test Steps

Build and Run MPU demo on STM32H7 board.

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.

Related Issue

PR# 942

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

@kar-rahul-aws kar-rahul-aws requested a review from a team as a code owner January 24, 2024 18:47
Copy link

codecov bot commented Jan 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8e664fc) 93.42% compared to head (279b672) 93.42%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #961   +/-   ##
=======================================
  Coverage   93.42%   93.42%           
=======================================
  Files           6        6           
  Lines        3194     3194           
  Branches      885      885           
=======================================
  Hits         2984     2984           
  Misses        103      103           
  Partials      107      107           
Flag Coverage Δ
unittests 93.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@kar-rahul-aws
Copy link
Member Author

These changes have already been included in PR# #942

@kar-rahul-aws kar-rahul-aws deleted the update_ARMv8M_uxStackDepth branch February 1, 2024 09:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant