[PATCH] D40324: [libcxx] [test] Resolve C1XX warning in new vector<bool>::size tests caused by assert(true)

Marshall Clow via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 21 17:32:05 PST 2017


mclow.lists added a comment.

D'Oh - that was a paste that went wrong.
You are correct that `assert(c.size() == 3)` is correct.

However, what I was *trying* to fix were the lines before.
Pushing a `3` into a `vector<bool>` is not the best idea L#34 and L#53 should both read `c.push_back(true);`


https://reviews.llvm.org/D40324





More information about the cfe-commits mailing list