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

Functions of rti.c moved to rti_lib.c to enable reuse #172

Merged
merged 6 commits into from
Mar 5, 2023
Merged

Conversation

Jakio815
Copy link
Collaborator

@Jakio815 Jakio815 commented Mar 1, 2023

This PR is about #170.

The original rti.c and rti.h #include each other files. I made a common lib_rti.c which includes rti_lib.h. rti.c with a main() function now only includes rti_lib.h.

The rti_lib.c is an exact copy of rti.c only excluding main(). rti_lib.h is also a copy of rti.h, and added declarations of functions in rti_lib.c. This is for reuse of the functions in rti_lib.c to make a rti_secure which is working on #143.

@Jakio815 Jakio815 changed the title Split up functions of rti.c to lib_rti.c for reuse of functions. Split up functions of rti.c to rti_lib.c for reuse of functions. Mar 1, 2023
Copy link
Member

@lhstrh lhstrh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@lhstrh lhstrh merged commit 0f598a7 into main Mar 5, 2023
@lhstrh lhstrh deleted the refactor-rti branch March 5, 2023 00:35
lhstrh added a commit that referenced this pull request Mar 6, 2023
This reverts commit 0f598a7, reversing
changes made to 4f78b42.
@lhstrh lhstrh changed the title Split up functions of rti.c to rti_lib.c for reuse of functions. Functions of rti.c moved to rti_lib.c to enable reuse Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants