[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