Skip to content

Commit

Permalink
lib/libc: rearrange for standard use of extern "C"
Browse files Browse the repository at this point in the history
Consistently place C++ use of extern "C" after all include directives,
within the negative branch of _ASMLANGUAGE if used.

Background from issue zephyrproject-rtos#17997:

Declarations that use C linkage should be placed within extern "C"
so the language linkage is correct when the header is included by
a C++ compiler.

Similarly #include directives should be outside the extern "C" to
ensure the language-specific default linkage is applied to any
declarations provided by the included header.

See: https://en.cppreference.com/w/cpp/language/language_linkage
Signed-off-by: Peter Bigot <[email protected]>
  • Loading branch information
pabigot authored and ioannisg committed Aug 19, 2019
1 parent 23fdb4b commit 849df51
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions lib/libc/minimal/include/time.h
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
#include <stdint.h>
#include <sys/_types.h>
#include <bits/restrict.h>
#include <sys/_timespec.h>

/* Minimal time.h to fulfill the requirements of certain libraries
* like mbedTLS and to support time APIs.
Expand Down Expand Up @@ -42,8 +43,6 @@ typedef _TIME_T_ time_t;
typedef _SUSECONDS_T_ suseconds_t;
#endif

#include <sys/_timespec.h>

/*
* Conversion between civil time and UNIX time. The companion
* localtime() and inverse mktime() are not provided here since they
Expand Down

0 comments on commit 849df51

Please sign in to comment.