[libcxx-commits] [PATCH] D96477: [libcxx] adds remaining callable concepts
Christopher Di Bella via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Feb 11 11:39:16 PST 2021
cjdb marked 2 inline comments as done.
cjdb added inline comments.
================
Comment at: libcxx/test/std/concepts/callable/functions.h:8
//===----------------------------------------------------------------------===//
-#ifndef CALLABLE_FUNCTIONS_H
-#define CALLABLE_FUNCTIONS_H
+#ifndef TEST_STD_CONCEPTS_LAND_CALLABLE_FUNCTIONS_H
+#define TEST_STD_CONCEPTS_LAND_CALLABLE_FUNCTIONS_H
----------------
curdeius wrote:
> `LAND_` should be removed, nope?
See @ldionne's request in D96230.
================
Comment at: libcxx/test/std/concepts/callable/predicate.compile.pass.cpp:94
+}
\ No newline at end of file
----------------
curdeius wrote:
> Nit: newline.
Done, but not pushing to remote since it's not worth activating CI for this.
================
Comment at: libcxx/test/std/concepts/callable/strictweakorder.compile.pass.cpp:102
+}
\ No newline at end of file
----------------
curdeius wrote:
> Newline.
Done, as above.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96477/new/
https://reviews.llvm.org/D96477
More information about the libcxx-commits
mailing list