Open zfields opened 7 months ago
Well, in such a large application consisting of quite some 3rd party code this is a bit of a pointless exercise. If you take a look at the error messages you'll see that quite some originate i.e. from the Renesas FSP layer. They would have to be fixed there first. As for -Wpedantic
errors originating from code under our control please feel free to provide a PR to address them. Just understand that if you enable -Wpedantic -Werror
there will very likely always be failing code pieces due to 3rd party software.
I'm not trying to point the finger at anyone, or make "pedantic" warnings seem super important. Furthermore, I fully understand that -Wpedantic
is an extreme configuration, and I understand many of them have to do with third-party code. That being said, it's is still worth cleaning up the things that can be cleaned, and may be under the scope of your control.
I'm more than happy to contribute and I will try to create PRs as I can :+1:, but to provide completely candid feedback -- my urge is to switch platforms entirely. I offer this only to say, when other people see such a significant amount of warnings generated, they may have a similar reaction.
Now that I know you are open to it, I will try to send PRs your way.
Now that I know you are open to it, I will try to send PRs your way.
Absolutely. Looking forward to getting those PRs :rocket:
I added
-Wpedantic
to theplatform.txt
file, and there are too many warnings to paste in this issue.These should be addressed to ensure the stability of the platform.