[PATCH] D61865: [clangd] improve help message for limit-results
This revision was automatically updated to reflect the committed changes. Closed by commit rL361388: [clangd] improve help message for limit-results (authored by kadircet, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D61865?vs=199295&id=200734#toc Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61865/new/ https://reviews.llvm.org/D61865 Files: clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp Index: clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp === --- clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp +++ clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp @@ -119,7 +119,7 @@ static llvm::cl::opt LimitResults( "limit-results", llvm::cl::desc("Limit the number of results returned by clangd. " - "0 means no limit."), + "0 means no limit. (default=100)"), llvm::cl::init(100)); static llvm::cl::opt RunSynchronously( Index: clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp === --- clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp +++ clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp @@ -119,7 +119,7 @@ static llvm::cl::opt LimitResults( "limit-results", llvm::cl::desc("Limit the number of results returned by clangd. " - "0 means no limit."), + "0 means no limit. (default=100)"), llvm::cl::init(100)); static llvm::cl::opt RunSynchronously( ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D61865: [clangd] improve help message for limit-results
kadircet added a comment. I can land it for you, you can learn more about commit access from https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61865/new/ https://reviews.llvm.org/D61865 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D61865: [clangd] improve help message for limit-results
umanwizard added a comment. @kadircet what are the next steps? This is my first diff in llvm. I'm not a project member so presumably someone else will have to land it for me. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61865/new/ https://reviews.llvm.org/D61865 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D61865: [clangd] improve help message for limit-results
kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61865/new/ https://reviews.llvm.org/D61865 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D61865: [clangd] improve help message for limit-results
umanwizard added a comment. Ping. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61865/new/ https://reviews.llvm.org/D61865 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D61865: [clangd] improve help message for limit-results
umanwizard created this revision. umanwizard added a reviewer: clang-tools-extra. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Make it clear that the default is 100. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D61865 Files: clang-tools-extra/clangd/tool/ClangdMain.cpp Index: clang-tools-extra/clangd/tool/ClangdMain.cpp === --- clang-tools-extra/clangd/tool/ClangdMain.cpp +++ clang-tools-extra/clangd/tool/ClangdMain.cpp @@ -117,7 +117,7 @@ static llvm::cl::opt LimitResults( "limit-results", llvm::cl::desc("Limit the number of results returned by clangd. " - "0 means no limit."), + "0 means no limit. (default=100)"), llvm::cl::init(100)); static llvm::cl::opt RunSynchronously( Index: clang-tools-extra/clangd/tool/ClangdMain.cpp === --- clang-tools-extra/clangd/tool/ClangdMain.cpp +++ clang-tools-extra/clangd/tool/ClangdMain.cpp @@ -117,7 +117,7 @@ static llvm::cl::opt LimitResults( "limit-results", llvm::cl::desc("Limit the number of results returned by clangd. " - "0 means no limit."), + "0 means no limit. (default=100)"), llvm::cl::init(100)); static llvm::cl::opt RunSynchronously( ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits