Removed function pointers from heap implementation

[esmakokten]

Summary of this Pull Request (PR)

In order to do stack analysis for components, function pointers are removed from heap impl. Added macro definition 'DECLARE_HEAP' to define heap with given update and compare functions. Added unit tests to test heap functionality.

Intent for your PR

Choose one (Mandatory):

• [ ] This PR is for a code-review and is intended to get feedback, but not to be pulled yet.
• [x] This PR is mature, and ready to be integrated into the repo.

Reviewers (Mandatory):

@gparmer @mbai1010 @spadek67424

Code Quality

As part of this pull request, I've considered the following:

Style:

• [ ] Comments adhere to the Style Guide (SG)
• [x] Spacing adhere's to the SG
• [x] Naming adhere's to the SG
• [ ] All other aspects of the SG are adhered to, or exceptions are justified in this pull request
• [ ] I have run the auto formatter on my code before submitting this PR (see doc/auto_formatter.md for instructions)

Code Craftsmanship:

• [x] I've made an attempt to remove all redundant code
• [x] I've considered ways in which my changes might impact existing code, and cleaned it up
• [x] I've formatted the code in an effort to make it easier to read (proper error handling, function use, etc...)
• [ ] I've commented appropriately where code is tricky
• [x] I agree that there is no "throw-away" code, and that code in this PR is of high quality

Testing

I've tested the code using the following test programs (provide list here):

• [ ] micro_booter
• [ ] unit_pingpong
• [x] sched_ping_pong
• [x] unit_heap
• [ ] unit_schedtests
• [ ] ...(add others here)

[spadek67424]

Hi @esmakokten @gparmer @mbai1010 , From my point, it is good to me. It fixs the function pointer of heap for me. I do not see the function pointer of heap anymore.

I am not sure about the file structure and coding style. Maybe somebody could have a look?

But I have an off-topic question is that do you miss a commit "7a0fe09d614f5ec7159fc2ecb72aec380172266e" from your branch?

Maybe you squash the commit?
I do not use PR a lot. Just want to make sure.

[esmakokten]

Hi @esmakokten @gparmer @mbai1010 , From my point, it is good to me. It fixs the function pointer of heap for me. I do not see the function pointer of heap anymore.

I am not sure about the file structure and coding style. Maybe somebody could have a look?

But I have an off-topic question is that do you miss a commit "7a0fe09d614f5ec7159fc2ecb72aec380172266e" from your branch?

Maybe you squash the commit? I do not use PR a lot. Just want to make sure.

Hi Ming-Hsien, thanks for the review. Hmm, I have two commits that will be integrated to main branch one of them is the "7a0fe09", I checked the PR and commits it seems correct to me, thanks!

[gparmer]

Thanks a ton! Looks great.