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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
