[PATCH] D26885: [libcxx] Fix max_size() across all containers
Eric Fiselier via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 22 17:28:00 PST 2016
EricWF added a comment.
This patch reminds me; The way we test our containers is ridiculous. We should have a single template that tests the container requirements, and each container test should just use that template.
Anyway. accepting before commit.
================
Comment at: test/std/containers/associative/map/map.access/max_size.pass.cpp:24
+
+template <class Alloc> size_t alloc_max_size(Alloc const &a) {
+ typedef std::allocator_traits<Alloc> AT;
----------------
mclow.lists wrote:
> How about a newline here? (after the `template<>`)
>
> And why not hoist this into "test_allocator.h"?
>
I'll change the `.clang-format` to add newlines after templates, and move this to test_allocator.h
https://reviews.llvm.org/D26885
More information about the cfe-commits
mailing list