This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] [test] Make istreambuf.iterator/types.pass.cpp more portable.
ClosedPublic

Authored by STL_MSFT on Jun 29 2016, 4:38 PM.

Details

Summary

Make istreambuf.iterator/types.pass.cpp more portable.

There are two issues here. First, istreambuf_iterator::pointer is unspecified (and MSVC uses a different type). Second, C++17 doesn't mandate std::iterator inheritance anymore (MSVC currently inherits, but we'll probably change this in the future).

These changes future-proof the test by inspecting the typedefs, and marking the pointer static_assert as libcxx-specific.

Diff Detail

Event Timeline

STL_MSFT updated this revision to Diff 62309.Jun 29 2016, 4:38 PM
STL_MSFT retitled this revision from to [libcxx] [test] Make istreambuf.iterator/types.pass.cpp more portable..
STL_MSFT updated this object.
STL_MSFT added reviewers: EricWF, mclow.lists.
STL_MSFT added a subscriber: cfe-commits.
EricWF accepted this revision.Jun 29 2016, 9:37 PM
EricWF edited edge metadata.
This revision is now accepted and ready to land.Jun 29 2016, 9:37 PM
EricWF closed this revision.Jun 29 2016, 9:38 PM

r274207.