This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGef8121b109ef: [Headers] Remove a space in NULL define (authored by aheejin).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159312/new/ https://reviews.llvm.org/D159312 Files: clang/lib/Headers/__stddef_null.h Index: clang/lib/Headers/__stddef_null.h =================================================================== --- clang/lib/Headers/__stddef_null.h +++ clang/lib/Headers/__stddef_null.h @@ -15,5 +15,9 @@ #define NULL 0 #endif #else -#define NULL ((void *)0) +// Don't add any whitespaces in ((void*)0) below! +// musl (https://www.musl-libc.org/) redefines `NULL` as such and redefinition +// with a different expression, even in terms of a single whitespace, causes a +// warning. +#define NULL ((void*)0) #endif
Index: clang/lib/Headers/__stddef_null.h =================================================================== --- clang/lib/Headers/__stddef_null.h +++ clang/lib/Headers/__stddef_null.h @@ -15,5 +15,9 @@ #define NULL 0 #endif #else -#define NULL ((void *)0) +// Don't add any whitespaces in ((void*)0) below! +// musl (https://www.musl-libc.org/) redefines `NULL` as such and redefinition +// with a different expression, even in terms of a single whitespace, causes a +// warning. +#define NULL ((void*)0) #endif
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
