[PATCH] D65320: [clangd] Fix background index not triggering on windows due to case mismatch.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 26 03:41:59 PDT 2019


sammccall planned changes to this revision.
sammccall marked an inline comment as done.
sammccall added a comment.

Sorry, this has a hideous bug that rebroadcasts CDBs over and over. Working on a fix...



================
Comment at: clang-tools-extra/clangd/GlobalCompilationDatabase.cpp:195
     if (Request.ShouldBroadcast && !SentBroadcast)
       CompilationDatabases[Result.PI.SourceRoot].SentBroadcast = true;
   }
----------------
this line is incorrect because the key should be lowercase


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65320/new/

https://reviews.llvm.org/D65320





More information about the cfe-commits mailing list