[PATCH] D13383: [clang] Add flag to DeclContext to distinguish between qualified and unqualified name lookups

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 6 06:25:52 PST 2015


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

With moving the definition of the Deinitializer class closer to its use (it should be defined within the function itself given the limited use), LGTM.


================
Comment at: clang/lib/Sema/SemaLookup.cpp:211
@@ +210,3 @@
+  public:
+    Deinitializer(const std::function<void()>& d): Deinit(d) {}
+    ~Deinitializer() { Deinit(); }
----------------
Formatting (the & goes with d).


http://reviews.llvm.org/D13383





More information about the cfe-commits mailing list