[PATCH] D58126: [clangd] Fix a lit-test.

2019-02-12 Thread Kadir Cetinkaya via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE353857: [clangd] Fix a lit-test. (authored by kadircet, committed by ). Changed prior to commit: https://reviews.llvm.org/D58126?vs=186475&id=186490#toc Repository: rCTE Clang Tools Extra CHANGES

[PATCH] D58126: [clangd] Fix a lit-test.

2019-02-12 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. 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 :-(

[PATCH] D58126: [clangd] Fix a lit-test.

2019-02-12 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov. Herald added a project: clang. Fixes https://bugs.llvm.org/show_bug.cgi?id=40593. Non-percent-encoded chars doesn't cause any problems on