-
Notifications
You must be signed in to change notification settings - Fork 236
Description
Is your feature request related to a problem? Please describe.
The CFE SB pipe ID table is not safe from aliasing or other issues.
Describe the solution you'd like
Redefine the CFE_SB_PipeID_t type to be a form of CFE_ES_ResourceID_t like many other resources have already been converted (mem pool handles, CDS blocks, etc).
Re-Use all the same management patterns of this structure.
Additional context
The only potential downside is that Resource IDs are defined as 32 bit values but Pipe IDs were only 8 bits. So this will make sizeof(CFE_SB_PipeId_t) into 4 instead of 1. But as long as apps are properly using the typedef and not assuming uint8 or otherwise depending on this being a single byte, this shouldn't be noticeable.
This is somewhat related to previous issue #100 - implementing this would be a good step in the right direction for that issue too.
Requester Info
Joseph Hickey, Vantage Systems, Inc.