Skip to content

Commit

Permalink
Merge pull request #1332 from nasa/integration-candidate
Browse files Browse the repository at this point in the history
osal Integration candidate: Caelum-rc4+dev28
  • Loading branch information
dzbaker authored Nov 14, 2022
2 parents 4567e8b + 8ca317e commit e98bb23
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 2 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Changelog

## Development Build: v6.0.0-rc4+dev151
- add BUGCHECK_VOID macro
- See <https://github.com/nasa/osal/pull/1323>

## Development Build: v6.0.0-rc4+dev147
- adjust timing on condvar tests
- See <https://github.com/nasa/osal/pull/1319>
Expand Down
9 changes: 9 additions & 0 deletions src/os/inc/osapi-macros.h
Original file line number Diff line number Diff line change
Expand Up @@ -136,4 +136,13 @@
*/
#define LENGTHCHECK(str, len, errcode) ARGCHECK(memchr(str, '\0', len), errcode)

/**
* @brief Bug-Check macro for void functions
*
* The basic BUGCHECK macro returns a value, which needs to be empty
* for functions that do not have a return value. In this case the
* second argument (errcode) is intentionally left blank.
*/
#define BUGCHECK_VOID(cond) BUGCHECK(cond, )

#endif /* OSAPI_MACROS_H */
2 changes: 1 addition & 1 deletion src/os/inc/osapi-version.h
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
/*
* Development Build Macro Definitions
*/
#define OS_BUILD_NUMBER 147
#define OS_BUILD_NUMBER 151
#define OS_BUILD_BASELINE "v6.0.0-rc4"

/*
Expand Down
2 changes: 1 addition & 1 deletion src/os/shared/src/osapi-printf.c
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ void OS_printf(const char *String, ...)
char msg_buffer[OS_BUFFER_SIZE];
int actualsz;

BUGCHECK((String) != NULL, )
BUGCHECK_VOID(String != NULL)

if (OS_SharedGlobalVars.GlobalState != OS_INIT_MAGIC_NUMBER)
{
Expand Down

0 comments on commit e98bb23

Please sign in to comment.