================
Comment at: lib/Sema/SemaDeclCXX.cpp:7335-7337
@@ -7329,4 +7334,5 @@
+
if (RequireMember && !isa<FieldDecl>(ND) && !isa<CXXMethodDecl>(ND) &&
!isa<TypeDecl>(ND))
return false;
----------------
Richard Smith wrote:
> Dinesh Dwivedi wrote:
> > Richard Smith wrote:
> > > Is this still necessary?
> > yes, I have added one test case which fails if we comment these lines. I
> > have kept these lines so that my change just filter out non class member if
> > RequiredMember is true but should not add any other type of class member
> > which it was not considering before my changes.
> I think your testcase shows that these lines are in fact incorrect -- if the
> function template were in a base class, rather than in an unrelated class,
> typo correction *should* find it.
No, in added test case, function template is not in base class.
http://reviews.llvm.org/D3051
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
