NoQ added inline comments.
================ Comment at: cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:73 + : II(nullptr), IsLookupDone(false), FuncName(FuncName), + RequiredArgs(RequiredArgs) {} ---------------- xazax.hun wrote: > NoQ wrote: > > Maybe `assert(FuncName.size() > 0)` here? > Oh, I see now. Yeah, that would make sense :) Do you want me to commit that > or will you? Because C++ checkers are receiving a lot of attention recently, i'd probably be looking into modifying this class to work with qualified names and overloads. You can commit though if you want it sooner :) Repository: rL LLVM https://reviews.llvm.org/D29884 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits