This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Add <experimental/system_error>
ClosedPublic

Authored by EricWF on Feb 2 2015, 12:12 PM.

Diff Detail

Event Timeline

EricWF updated this revision to Diff 19176.Feb 2 2015, 12:12 PM
EricWF retitled this revision from to [libcxx] Add <experimental/system_error>.
EricWF updated this object.
EricWF edited the test plan for this revision. (Show Details)
EricWF added a subscriber: Unknown Object (MLST).
mclow.lists accepted this revision.Feb 3 2015, 7:47 AM
mclow.lists edited edge metadata.

LGTM. I think the whole "static assertion on three lines" coding style is hard(er) to read, but the checks are there and are fine.

This revision is now accepted and ready to land.Feb 3 2015, 7:47 AM

I think @chandlerc said the hardest part of C++ is where to split the line. I'll try and come up with something better in the tests without violating the 80 col limit.

EricWF updated this revision to Diff 19225.Feb 3 2015, 8:03 AM
EricWF edited edge metadata.

I let clang-format take a pass at it and cleaned up what came out. It still doesn't look nice but it is better(?).

EricWF closed this revision.Feb 3 2015, 8:05 AM

Looks good to me.

test/std/experimental/utilities/syserror/header.system_error.synop/is_error_condition_enum.pass.cpp
24

You know, if I was writing this, I would align stuff to make it clear what's the same and what's different.

static_assert( ex::is_error_condition_enum_v<std::errc> ==
              std::is_error_condition_enum  <std::errc>::value, "" );

But that might just be me indulging my OCD.