Skip to content

Commit 44e956f

Browse files
aheejinAndrés Villegas
authored andcommitted
[Headers] Remove musl-related comment about NULL
This removes a comment added in D159312, which warned people to not re-add a whitespace in the `((void*)0))` expression. After discussions happened in D159312, it doesn't seem like a permanent solution. While I'd like to keep the whitespace removed for now, given that at least it can be a band-aid to some users who use musl and clang's `stddef.h` at the same time, it seems the usage of them together is not something that's officially supported, and I should not be implying this should be the permanent solution by saying so in the comments. Reviewed By: aaron.ballman, ributzka Differential Revision: https://reviews.llvm.org/D159383
1 parent bf4dc24 commit 44e956f

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

clang/lib/Headers/__stddef_null.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,5 @@
1515
#define NULL 0
1616
#endif
1717
#else
18-
// Don't add any whitespaces in ((void*)0) below!
19-
// musl (https://www.musl-libc.org/) redefines `NULL` as such and redefinition
20-
// with a different expression, even in terms of a single whitespace, causes a
21-
// warning.
2218
#define NULL ((void*)0)
2319
#endif

0 commit comments

Comments
 (0)