This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Fix incorrect guard against the presence of wide characters
ClosedPublic

Authored by ldionne on Nov 28 2022, 8:50 AM.

Details

Summary

TEST_HAS_NO_WIDE_CHARACTERS should only be used in the tests.

Diff Detail

Event Timeline

ldionne created this revision.Nov 28 2022, 8:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 28 2022, 8:50 AM
ldionne requested review of this revision.Nov 28 2022, 8:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 28 2022, 8:50 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Nov 28 2022, 10:46 AM
Mordante added a subscriber: Mordante.

Nice catch, seems like a copy-paste bug. LGTM!

This revision is now accepted and ready to land.Nov 28 2022, 10:46 AM
This revision was landed with ongoing or failed builds.Nov 28 2022, 2:34 PM
This revision was automatically updated to reflect the committed changes.