[PATCH] D57944: [clangd] Fix an assertion in TypoCorrection.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 8 05:11:05 PST 2019


hokein accepted this revision.
hokein added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clangd/IncludeFixer.cpp:195
 
-    assert(S && "Enclosing scope must be set.");
-
     UnresolvedName Unresolved;
     Unresolved.Name = Typo.getAsString();
----------------
nit: move the `Unresolved` below the `if (!SpecifiedScope && !S)`.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D57944





More information about the cfe-commits mailing list