Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Roll-up of general bplib improvements and clean-up #63

Merged
merged 3 commits into from
Jan 21, 2022

Conversation

jphickey
Copy link
Collaborator

Addresses several issues:

Fixes #60:

Qualify all pointers to objects which should typically not be modified
as "const".  Also changes use of "int" type to "size_t" where the value
indicates the size of an object in memory

Fixes #61:

Adds a "CMakeLists.txt" file such that bplib may be built as a CFE component.

The classic Makefile is preserved for the general-purpose build.  However
this could also be replaced with the CMake build if so desired.

Fixes #62:

Updates the "os/cfe.c" and "store/file_cfe.c" adapater layers which
are intended for integrating bplib into a CFE-based environment.

With these updates, the adapters should work with the current version
of the CFE framework.

Qualify all pointers to objects which should typically not be modified
as "const".  Also changes use of "int" type to "size_t" where the value
indicates the size of an object in memory.
Updates the "os/cfe.c" and "store/file_cfe.c" adapater layers which
are intended for integrating bplib into a CFE-based environment.

With these updates, the adapters should work with the current version
of the CFE framework.
Adds a "CMakeLists.txt" file such that bplib may be built as a CFE component.

The classic Makefile is preserved for the general-purpose build.  However
this could also be replaced with the CMake build if so desired.
@jphickey jphickey merged commit bd7164a into nasa:master Jan 21, 2022
@jphickey jphickey deleted the jph-bplib-issues-20220120 branch January 24, 2022 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant