[PATCH] D53016: [clangd] Mark colon as a safe character when percent-encoding.

2018-10-09 Thread Kadir Cetinkaya via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL344033: [clangd] Mark colon as a safe character when percent-encoding. (authored by kadircet, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D5301

[PATCH] D53016: [clangd] Mark colon as a safe character when percent-encoding.

2018-10-09 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet updated this revision to Diff 168771. kadircet added a comment. - Add comments. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53016 Files: clangd/URI.cpp unittests/clangd/URITests.cpp Index: unittests/clangd/URITests.cpp =

[PATCH] D53016: [clangd] Mark colon as a safe character when percent-encoding.

2018-10-09 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clangd/URI.cpp:94 case '/': // '/' is only reserved when parsing. + case ':': return false; add a comment: ':' is only reserve

[PATCH] D53016: [clangd] Mark colon as a safe character when percent-encoding.

2018-10-09 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. Also change output of percent-encoding to use upper-case letters. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53016