diff options
| author | Donald Sharp <donaldsharp72@gmail.com> | 2022-12-01 11:01:45 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-12-01 11:01:45 -0500 |
| commit | 5eb310087b200f81acc94beb50e9aea8c09eaa22 (patch) | |
| tree | 53974bbd89ff30c622cafa5910f1cd1afe5873b2 | |
| parent | 0ec0fee2795fab28daf107f6aec3c3871cded87d (diff) | |
| parent | a0823246493f72cf87a04595b219cf34dd976ad4 (diff) | |
Merge pull request #12397 from opensourcerouting/vscode-intro
lib: handle visual studio code C/C++ extension
| -rw-r--r-- | lib/compiler.h | 39 | ||||
| -rw-r--r-- | lib/prefix.h | 16 |
2 files changed, 39 insertions, 16 deletions
diff --git a/lib/compiler.h b/lib/compiler.h index bf443906eb..514f0cee6e 100644 --- a/lib/compiler.h +++ b/lib/compiler.h @@ -72,6 +72,16 @@ extern "C" { #endif #endif +#ifdef __INTELLISENSE__ +/* + * Fix Visual Studio Code error: attribute "constructor" does not take + * arguments. + * + * Caused by the macro `DEFINE_MTYPE_ATTR` in `lib/memory.h`. + */ +#pragma diag_suppress 1094 +#endif /* __INTELISENSE__ */ + #if __has_attribute(hot) # define _OPTIMIZE_HOT __attribute__((hot)) #else @@ -411,6 +421,35 @@ _Static_assert(sizeof(_uint64_t) == 8 && sizeof(_int64_t) == 8, #endif /* !__cplusplus */ #endif /* !_FRR_ATTRIBUTE_PRINTFRR */ +/* helper to get type safety/avoid casts on calls + * (w/o this, functions accepting all prefix types need casts on the caller + * side, which strips type safety since the cast will accept any pointer + * type.) + */ +#ifndef __cplusplus +#define prefixtype(uname, typename, fieldname) typename *fieldname; +#define TRANSPARENT_UNION __attribute__((transparent_union)) +#else +#define prefixtype(uname, typename, fieldname) \ + typename *fieldname; \ + uname(typename *x) \ + { \ + this->fieldname = x; \ + } +#define TRANSPARENT_UNION +#endif + +#ifdef __INTELLISENSE__ +/* + * Fix Visual Studio Code error: argument of type "struct prefix *" is + * incompatible with parameter of type "union prefixptr". + * + * This is caused by all functions having the transparent unions in the + * prototype. Example: `prefixptr` and `prefixconstptr` from `lib/prefix.h`. + */ +#pragma diag_suppress 167 +#endif /* __INTELISENSE__ */ + #ifdef __cplusplus } #endif diff --git a/lib/prefix.h b/lib/prefix.h index c67656cfd1..7de8d7903e 100644 --- a/lib/prefix.h +++ b/lib/prefix.h @@ -301,22 +301,6 @@ struct prefix_sg { struct in_addr grp; }; -/* helper to get type safety/avoid casts on calls - * (w/o this, functions accepting all prefix types need casts on the caller - * side, which strips type safety since the cast will accept any pointer - * type.) - */ -#ifndef __cplusplus -#define prefixtype(uname, typename, fieldname) \ - typename *fieldname; -#define TRANSPARENT_UNION __attribute__((transparent_union)) -#else -#define prefixtype(uname, typename, fieldname) \ - typename *fieldname; \ - uname(typename *x) { this->fieldname = x; } -#define TRANSPARENT_UNION -#endif - union prefixptr { prefixtype(prefixptr, struct prefix, p) prefixtype(prefixptr, struct prefix_ipv4, p4) |
