This is an archive of the discontinued LLVM Phabricator instance.

[libc++][test] Fix unused variable warning in string_view tests
ClosedPublic

Authored by jloser on Jun 2 2022, 11:13 AM.

Details

Summary

In 6423a9f0ec8ba70049ea76e7bcfc9a9d1a54e826, I accidentally thought this was
getting tested, but these variables are unused. Just remove the lines instead of
leaving them commented out.

Diff Detail

Event Timeline

jloser created this revision.Jun 2 2022, 11:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2022, 11:13 AM
jloser requested review of this revision.Jun 2 2022, 11:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2022, 11:13 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Jun 2 2022, 11:25 AM

Thanks! LGTM after addressing the review comments.

libcxx/test/std/strings/string.view/string.view.template/ends_with.ptr.pass.cpp
67–68

Remove these too please.

libcxx/test/std/strings/string.view/string.view.template/starts_with.ptr.pass.cpp
67–68

Remove these too please.

This revision is now accepted and ready to land.Jun 2 2022, 11:25 AM
This revision was landed with ongoing or failed builds.Jun 2 2022, 12:34 PM
This revision was automatically updated to reflect the committed changes.