This is an archive of the discontinued LLVM Phabricator instance.

libc++ locale: Disable cat*() on CloudABI. Also make tiny cleanups.
ClosedPublic

Authored by ed on Mar 9 2015, 4:16 AM.

Details

Summary

On a new platform that I am working on (https://github.com/NuxiNL/cloudlibc) I am not implementing the cat{open,close,gets}() API, just like Android, Newlib, etc.

Instead of adding yet another operating system name to the #ifs, introduce _LIBCPP_HAS_CATOPEN in include/__config. Also adjust the code to only pull in nl_types.h when _LIBCPP_HAS_CATOPEN is set. We only needed this header for the cat*() API.

Diff Detail

Repository
rL LLVM

Event Timeline

ed updated this revision to Diff 21478.Mar 9 2015, 4:16 AM
ed retitled this revision from to libc++ locale: Disable cat*() on CloudABI. Also make tiny cleanups..
ed updated this object.
ed edited the test plan for this revision. (Show Details)
ed added a reviewer: mclow.lists.
ed set the repository for this revision to rL LLVM.
ed added a subscriber: Unknown Object (MLST).
ed updated this revision to Diff 21482.Mar 9 2015, 6:26 AM
ed updated this object.
ed removed rL LLVM as the repository for this revision.
mclow.lists accepted this revision.Mar 11 2015, 9:17 AM
mclow.lists edited edge metadata.

This looks OK to me.

This revision is now accepted and ready to land.Mar 11 2015, 9:17 AM
This revision was automatically updated to reflect the committed changes.