[cfe-commits] PATCH: Add support for C++ namespace-aware typo correcting

Kaelyn Uhrain rikka at google.com
Tue Feb 22 11:21:27 PST 2011


This patch implements checking available C++ namespaces when trying to find
typo corrections for unknown identifiers.  There are currently 4 broken unit
tests (CXX/basic/basic.lookup/basic.lookup.argdep/p4.cpp,
CXX/class/class.local/p1.cpp, FixIt/typo.cpp, and
SemaObjC/synth-provisional-ivars.m) which I'm fairly sure are cases where
the tests need to be updated to reflect the new behavior instead of being
indicative of actual code breakage, but I'm not 100% certain.  The new unit
test, test/Sema/missing-namespace-qualifier-typo-corrections.cpp, is a good
demonstration of the improved suggestions provided by this patch as most of
the errors found by "clang -fsyntax-only" lack suggestions without this
patch..

Cheers,
Kaelyn
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110222/bca69748/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-namespace-aware-typo-corrections.diff
Type: text/x-diff
Size: 26232 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110222/bca69748/attachment.diff>


More information about the cfe-commits mailing list