[libc-commits] [PATCH] D77279: [libc] Add strlen implementation.
Siva Chandra via Phabricator via libc-commits
libc-commits at lists.llvm.org
Fri Apr 3 11:53:37 PDT 2020
sivachandra added inline comments.
================
Comment at: libc/src/string/strlen.h:13
+#include "include/string.h"
+#include <stddef.h> // size_t
----------------
sivachandra wrote:
> Do you still need to include `stddef.h` if you include `include/string.h`? In my patches, I have included the public header iff required. For example, the public header defines a header which is required by the API declared here. So, keeping `stddef.h` and removing `include/string.h` is also fine. Whatever you choose, share your thoughts so that I can learn if I am missing something.
I mean't to say, "For example, if the public header defines a *struct* which is required by the API declared here."
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77279/new/
https://reviews.llvm.org/D77279
More information about the libc-commits
mailing list