aaron.ballman added a comment. I'm uncertain whether this change is good or not (Richard is more likely to have thoughts on that), but the patch is missing tests.
================ Comment at: clang/lib/Sema/SemaLookup.cpp:208 @@ +207,3 @@ + + class Deinitializer { + std::function<void()> Deinit; ---------------- This should be a local class defined more closely to its usage. http://reviews.llvm.org/D13383 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits