[PATCH] D34137: [clangd] Add priority to completion item sort text

2017-06-15 Thread Krasimir Georgiev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL305454: [clangd] Add priority to completion item sort text (authored by krasimir). Changed prior to commit: https://reviews.llvm.org/D34137?vs=102522&id=102646#toc Repository: rL LLVM https://review

[PATCH] D34137: [clangd] Add priority to completion item sort text

2017-06-14 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 102522. krasimir added a comment. - Address review comments https://reviews.llvm.org/D34137 Files: clangd/ClangdUnit.cpp test/clangd/authority-less-uri.test test/clangd/completion.test Index: test/clangd/completion.test =

[PATCH] D34137: [clangd] Add priority to completion item sort text

2017-06-14 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov added inline comments. Comment at: clangd/ClangdUnit.cpp:156 assert(CCS->getTypedText()); +CCS->getPriority(); Item.kind = getKind(Result.CursorKind); This is a no-op, commited incidentally? Comment at: c