>+ if ((Mode == CTK_ErrorRecovery) && getLangOpts().Modules &&
>+ getLangOpts().ModulesSearchAll) {
>+ if (PP.getModuleLoader().lookupMissingImports(Typo->getName(),
>+ TypoName.getLocStart())) {
>+ TypoCorrection TC(TypoName.getName(), (NestedNameSpecifier *)0, 0);
>+ TC.setCorrectionRange(SS, TypoName);
>+ TC.setRequiresImport(true);
>
>This TC variable is unused; remove it?
>
>I suspect TC was intended to be added to Consumer so that it participates in
>typo correction as a candidate... John, is that a correct >assumption? If not
>then this block of code does nothing beyond whatever side effects
>.lookupMissingImports() may have, and the >TC variable should be removed.
Thanks for finding this! Fixed in r208259.
-John
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
