Ping? I'd like at least a little feedback before submitting this patch since it is the first time I've done anything with the parser code.
On Tue, Sep 13, 2011 at 5:22 PM, Kaelyn Uhrain <[email protected]> wrote: > FYI, this patch allows for typo correction in what is from a user > perspective a reasonable case where clang should suggest a correction but > does not. Right now in: > > namespace test { > class FizBinHelper {}; > } > > void foo() { > test::FizBinHelper *x; > x = new test::FizbinHelper; > } > > clang will just say "expected a type" instead of suggesting correcting > test::FizbinHelper to test::FizBinHelper (fixing a simple case-mismatch > typo). > > Just in case it wasn't clear what the context of the change is from the > changes to the test cases. ;) > > Cheers, > Kaelyn > > > On Tue, Sep 13, 2011 at 5:16 PM, <[email protected]> wrote: > >> Reviewers: cfe-commits_cs.uiuc.edu, >> >> >> >> Please review this at >> http://codereview.appspot.com/**5014044/<http://codereview.appspot.com/5014044/> >> >> Affected files: >> M include/clang/Parse/Parser.h >> M include/clang/Sema/Sema.h >> M lib/Parse/ParseDecl.cpp >> M lib/Parse/ParseExprCXX.cpp >> M lib/Parse/Parser.cpp >> M lib/Sema/SemaDecl.cpp >> M test/SemaCXX/missing-**namespace-qualifier-typo-**corrections.cpp >> >> >> >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
