search

nasa / sample_app

The Core Flight System (cFS) Sample App (sample_app)
Apache License 2.0
49 stars 43 forks source link

Fix #153, apply CFE_SB_ValueToMsgId where required #154

Closed jphickey closed 3 years ago

jphickey commented 3 years ago

Describe the contribution Whenever an integer value is used as a CFE_SB_MsgId_t, it should go through the explicit conversion using the supplied inline function.

Fixes #153

Testing performed Build and sanity check CFE

Expected behavior changes Allows SAMPLE_APP to be built when CFE_SB_MsgId_t is an opaque/non-integer type. None with default config (where CFE_SB_MsgId_t is an integer type).

System(s) tested on Ubuntu

Contributor Info - All information REQUIRED for consideration of pull request Joseph Hickey, Vantage Systems, Inc.

astrogeco commented 3 years ago

@ejtimmon do we need to do this for the GSFC Apps?

astrogeco commented 3 years ago

CCB:2021-09-21 APPROVED