This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Fix a lit-test.
ClosedPublic

Authored by kadircet on Feb 12 2019, 8:06 AM.

Details

Summary

Fixes https://bugs.llvm.org/show_bug.cgi?id=40593.
Non-percent-encoded chars doesn't cause any problems on the input-side since we
assume everything after authority section is data and don't interpret them as
delimeters.

Diff Detail

Event Timeline

kadircet created this revision.Feb 12 2019, 8:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 12 2019, 8:07 AM
sammccall accepted this revision.Feb 12 2019, 8:21 AM

Hmm, this removes a useful assertion, and still leaves us sending invalid (or at least non-canonical) URIs in the input.
But I don't see a better fix really - lit tests are painful :-(

This revision is now accepted and ready to land.Feb 12 2019, 8:21 AM
This revision was automatically updated to reflect the committed changes.