Macro usage cleanup - Githubissues

HTRamsey commented 10 months ago

Macro usage cleanup

Description

Remove defined checks as these macros are always defined
Remove redundant definitions of macros that are always defined
Move Static Assert definition to default macros so that it can be used more extensively for compile time checks

Test Steps

Checklist:

[x] 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

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

HTRamsey commented 10 months ago

/bot run formatting

Skptak commented 10 months ago

/bot run formatting

tony-josi-aws commented 10 months ago

Thanks @HTRamsey for taking time to contribute to FreeRTO+TCP, the changes look good. As v4.1 is around the corner we might have to include this change once v4.1 is released.

Skptak commented 10 months ago

Thanks @HTRamsey for raising the PR and for answering my questions, this looks good to me as well so I've approved it!

FreeRTOS / FreeRTOS-Plus-TCP

Macro usage cleanup #1080

Description

Test Steps

Checklist:

Related Issue