[PATCH] D49967: [clangd] Add command-line option to suppress the space and the circular dot prepended in a completion label.

2018-07-29 Thread Raoul Wols via Phabricator via cfe-commits
rwols closed this revision. rwols added a comment. Oh... I should have used `arc land` instead of `svn commit`. Here's the commit: https://reviews.llvm.org/rCTE338223 Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D49967 ___

[PATCH] D49967: [clangd] Add command-line option to suppress the space and the circular dot prepended in a completion label.

2018-07-29 Thread Raoul Wols via Phabricator via cfe-commits
rwols updated this revision to Diff 157894. rwols added a comment. Avoid double negative for command line option Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D49967 Files: clangd/tool/ClangdMain.cpp Index: clangd/tool/ClangdMain.cpp

[PATCH] D49967: [clangd] Add command-line option to suppress the space and the circular dot prepended in a completion label.

2018-07-29 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. Makes sense, thanks! Comment at: clangd/tool/ClangdMain.cpp:161 +static llvm::cl::opt NoHeaderInsertDecorators( +"no-header-insert-decorators", +

[PATCH] D49967: [clangd] Add command-line option to suppress the space and the circular dot prepended in a completion label.

2018-07-29 Thread Raoul Wols via Phabricator via cfe-commits
rwols created this revision. rwols added reviewers: sammccall, ilya-biryukov, ioeric. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay. Some LSP clients (e.g. Sublime Text) assume that the label and the completion trigger have a common prefix. This assumption is broken by