[clang-tools-extra] r351809 - [clangd] Fix the `-Wtype-limits` warning, NFC

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 22 04:21:25 PST 2019


Author: hokein
Date: Tue Jan 22 04:21:25 2019
New Revision: 351809

URL: http://llvm.org/viewvc/llvm-project?rev=351809&view=rev
Log:
[clangd] Fix the `-Wtype-limits` warning, NFC

The assertion is always true, and triggers a compiler warning, so remove it.

Modified:
    clang-tools-extra/trunk/clangd/index/Merge.cpp

Modified: clang-tools-extra/trunk/clangd/index/Merge.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/index/Merge.cpp?rev=351809&r1=351808&r2=351809&view=diff
==============================================================================
--- clang-tools-extra/trunk/clangd/index/Merge.cpp (original)
+++ clang-tools-extra/trunk/clangd/index/Merge.cpp Tue Jan 22 04:21:25 2019
@@ -102,7 +102,6 @@ void MergedIndex::refs(const RefsRequest
     Callback(O);
     --Remaining;
   });
-  assert(Remaining >= 0);
   if (Remaining == 0)
     return;
   // We return less than Req.Limit if static index returns more refs for dirty




More information about the cfe-commits mailing list