Merged
Conversation
These are given funky names by gcc, like csp_buffer_pool.2, which make them difficult for static analysis tools to find and put into context. Moving to file scope is only a slight scope increase which can be tolerated for the benefits of static analysis.
For remote parameters use param_define_remote instead
also remove include vmem.h as its not used
This conflicts with some users having const region and others non const region
User may not always double check his param is found after param_list_find so add this as an added safety aginst users
The init, configure and periodic calls, together with shall publish hook was just too much traffic in and out of the library. This is more efficient and easy to read code to perform all in userspace code.
If a parameters physical address is not aligned to the type it will assert. If arrays are used, and the size of the array step is not aligned to the type, it will also assert.
limits.h uses a gcc feature called include_next, this one is warned about when using pedantic warning level. There is no way to ignore this one, so we need to go down to level 2.
This avoids integer constant compiler warning
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This branch started since maing param_t const, changes the interface.
It further collects a number of improvements to memory and stack usage.