-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Fix cast alignment warning in heap_4.c and heap_5.c #771
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
Fix cast alignment warning in heap_4.c and heap_5.c #771
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #771 +/- ##
=======================================
Coverage 94.35% 94.35%
=======================================
Files 6 6
Lines 2446 2446
Branches 598 598
=======================================
Hits 2308 2308
Misses 85 85
Partials 53 53
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
edbeb62
Hi @kar-rahul-aws , I have had to revert this PR on my platform (Zynq 7000 ARM CA9 ) it causes my system to fail the very first call to malloc asserting no memory when the scheduler is started. I don't have much time today to look into it but I may be able to look into it later in the week if you are unable to address it in the meantime... cc @aggarg Kind Regards, Pete |
Hey @pete-pjb, Thank you for letting us know. Looking into it! |
This PR should fix the issue - #781. Thanks again! |
It does indeed fix the issue, thank you @aggarg 🙂 cc @tony-josi-aws |
Description
Without this change, the code produces the following warning when compiled with
-Wcast-align
flag:Test Steps
Checklist:
Related Issue
https://forums.freertos.org/t/freertos-heap-4-c-cast-error/18105
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.