Closed LasseRosenow closed 2 days ago
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
This should be done now and is ready for review
Memory usage after merging this PR will be:
The compiler complains about the zero length arrays that the macros can generate. I think we can solve it with more macro
Yeah actually the RIOT compiler is quite strict. I have already disabled a few rules to make it run. I think for now I could disable this rule also to make the ci run for now, but we should fix it in another PR
Memory usage after merging this PR will be:
Yeah, it appears that GNU C allows it, but not ISO C. We are currently using several GCC extensions so technically we arent ISO C compatible. That could be an issue
Memory usage after merging this PR will be:
Memory usage after merging this PR will be:
Coverage after merging run-riot-examples-in-ci into main will be
75.14% |
---|
Nice the test now runs successfully :) I opened #105 to not forget about the compiler issues, or to at least move the discussion there if we want to be ISO-C compatible or not etc.
Sorry I accidentally merged using the wrong button and moved all the commits to the main branch. I fixed it now
Closes #101