[PATCH] D40707: [libcxx] Fix basic_stringbuf constructor
Marshall Clow via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 11 10:57:21 PST 2017
mclow.lists added a comment.
This looks good to me; with a couple of nits.
================
Comment at: test/std/input.output/string.streams/stringbuf/stringbuf.cons/default.pass.cpp:26
+ {
+ assert(this->eback() == 0);
+ assert(this->gptr() == 0);
----------------
Not zero, please. `nullptr` or `NULL` .
`nullptr` would be better, but we can't assume c++11 here, so I guess it's `NULL`.
https://reviews.llvm.org/D40707
More information about the cfe-commits
mailing list