Scrub for include <> vs "" use (<> should be system only) #868

skliper · 2021-03-10T19:28:55Z

Is your feature request related to a problem? Please describe.
<> used on non-system header includes. Example:

osal/src/os/shared/inc/os-shared-dir.h

Line 32 in ead5723

#include <os-shared-globaldefs.h>

Describe the solution you'd like
Full scrub/fix

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC, OSAL code review

Fix #868, Use quotes for local includes

Fix nasa#858, use osal id typedef

skliper added enhancement CFS-38 labels Mar 10, 2021

skliper added this to the 6.0.0 milestone Mar 10, 2021

skliper self-assigned this Mar 11, 2021

skliper added a commit to skliper/osal that referenced this issue Mar 11, 2021

Fix nasa#868, Use quotes for local includes

98d4af0

skliper mentioned this issue Mar 11, 2021

Fix #868, Use quotes for local includes #892

Merged

skliper added a commit to skliper/osal that referenced this issue Mar 12, 2021

Fix nasa#868, Use quotes for local includes

5980693

astrogeco mentioned this issue Mar 17, 2021

osal Integration Candidate: 2021-03-23 #917

Merged

astrogeco added a commit that referenced this issue Mar 17, 2021

Merge pull request #892 from skliper/fix868-include_quotes

c332d9b

Fix #868, Use quotes for local includes

astrogeco closed this as completed in #917 Mar 22, 2021

jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022

Merge pull request nasa#868 from jphickey/fix-858-osal-id-type

bc48dc0

Fix nasa#858, use osal id typedef

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Scrub for include <> vs "" use (<> should be system only) #868

Scrub for include <> vs "" use (<> should be system only) #868

skliper commented Mar 10, 2021

Scrub for include <> vs "" use (<> should be system only) #868

Scrub for include <> vs "" use (<> should be system only) #868

Comments

skliper commented Mar 10, 2021