Skip to content
New issue

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

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

Already on GitHub? # to your account

Add C++ header guards #2412

Closed
wants to merge 1 commit into from
Closed

Conversation

irowebbn
Copy link
Contributor

@irowebbn irowebbn commented Jul 21, 2023

Checklist (Please check before submitting)

Describe the contribution
Adds header guards to public includes so CFE can be called from C++ translation units

Testing performed

  • TODO

Expected behavior changes

  • No impact to behavior
  • Allow for easy calling from C++ translation units

System(s) tested on

  • TODO

Additional context
Addresses nasa/cFS#688 for CFE

Contributor Info - All information REQUIRED for consideration of pull request
Isaac Rowe, NASA JSC, Jacobs Technology

@irowebbn irowebbn marked this pull request as ready for review July 21, 2023 20:57
@dzbaker
Copy link
Collaborator

dzbaker commented Nov 30, 2023

CCB 30 November 2023: Closing as we don't test C++ support.

@dzbaker dzbaker closed this Nov 30, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants