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

Process cFE Application Developers Guide with Doxygen to allow linkage to code/files #202

Open
skliper opened this issue Sep 30, 2019 · 7 comments
Labels
docs This change only affects documentation. enhancement

Comments

@skliper
Copy link
Contributor

skliper commented Sep 30, 2019

As discussed at the 5-24-16 CCB meeting (reference cFS Systems Engineering Handbook discussion in the minutes), the cFE application developers guide contains a significant amount of API references. Indirectly referencing the cFS API's in an external document is prone to error. The API information should be referenced directly from the code. It is highly recommended to move the Application Developers Guide to the cFE Doxygen User’s Guide.

@skliper skliper added this to the 6.7.1 milestone Sep 30, 2019
@skliper skliper self-assigned this Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Imported from trac issue 171. Created by sstrege on 2016-07-08T12:27:02, last modified: 2019-07-03T12:48:08

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by sstrege on 2016-09-15 20:50:25:

This guide should also include information on how to write endian neutral code:

On Thursday, September 15, 2016 7:10:10 PM EDT Scott, Michael S. (ARC-TI)[SGT, INC] wrote:

In our project we used this:

INCLUDE(TestBigEndian)

TEST_BIG_ENDIAN(BIGENDIAN)
IF(${BIGENDIAN})
ADD_DEFINITIONS(-DBIGENDIAN)
ENDIF(${BIGENDIAN})

This is the correct way to do it in cmake, but as Joe said there isn't really much reason to know this in cmake.

With regard to Joe's email, a "better" way to do this in C is:

#define IS_LITTLE_ENDIAN (((union {unsigned x; char c;}){1}).c)

This is known at compile time, which is nice. So you can even do something like

#if IS_LITTLE_ENDIAN
...
#else
...
#endif

Steven

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by glimes on 2016-11-08 14:17:08:

Current crop of cfe-next are all going into CFE 6.6

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jhageman on 2019-07-03 12:48:08:

Moved unfinished 6.6.1 issues to next minor release

@skliper skliper removed their assignment Sep 30, 2019
@skliper skliper modified the milestones: 6.7.1, 6.7.0, 6.8.0 Sep 30, 2019
@skliper skliper removed this from the 6.8.0 milestone Nov 4, 2019
@skliper
Copy link
Contributor Author

skliper commented Nov 4, 2019

Note #361 added a markdown version, but conversion to doxygen would make it linkable and reduce duplicated documentation.

@jphickey
Copy link
Contributor

jphickey commented Nov 4, 2019

Note #361 added a markdown version, but conversion to doxygen would make it linkable and reduce duplicated documentation.

Can't the markdown file itself just be included in the doxygen? Doxygen supports using markdown files and including that in the generated HTML, and terms should be cross-linked as expected.

@skliper
Copy link
Contributor Author

skliper commented Nov 4, 2019

Absolutely, although I'd suggest another scrub in that context to remove duplicated documentation.

@skliper skliper added the docs This change only affects documentation. label May 12, 2020
@skliper skliper changed the title cFE Application Developers Guide Should be Doxygen/Markdown Based Process cFE Application Developers Guide with Doxygen to allow linkage to code/files Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change only affects documentation. enhancement
Projects
None yet
Development

No branches or pull requests

2 participants