Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10)

Unified Diff: tools/clang/blink_gc_plugin/tests/traceimpl.h

Issue 2655933002: blink_gc_plugin: retire overloaded traceImpl detection and handling. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/clang/blink_gc_plugin/tests/traceimpl.h
diff --git a/tools/clang/blink_gc_plugin/tests/traceimpl.h b/tools/clang/blink_gc_plugin/tests/traceimpl.h
index 8cb51b1820492be9a5cd9a0495041d44c08254b1..c2a93be536f6c5087ea253c93b3b68b2ce119f6e 100644
--- a/tools/clang/blink_gc_plugin/tests/traceimpl.h
+++ b/tools/clang/blink_gc_plugin/tests/traceimpl.h
@@ -16,12 +16,7 @@ class X : public GarbageCollected<X> {
class TraceImplInlined : public GarbageCollected<TraceImplInlined> {
public:
- void Trace(Visitor* visitor) { TraceImpl(visitor); }
-
- template <typename VisitorDispatcher>
- void TraceImpl(VisitorDispatcher visitor) {
- visitor->Trace(x_);
- }
+ void Trace(Visitor* visitor) { visitor->Trace(x_); }
private:
Member<X> x_;
@@ -30,8 +25,6 @@ class TraceImplInlined : public GarbageCollected<TraceImplInlined> {
class TraceImplExtern : public GarbageCollected<TraceImplExtern> {
public:
void Trace(Visitor* visitor);
- template <typename VisitorDispatcher>
- inline void TraceImpl(VisitorDispatcher);
private:
Member<X> x_;
@@ -44,21 +37,13 @@ class Base : public GarbageCollected<Base> {
class TraceImplBaseInlined : public Base {
public:
- void Trace(Visitor* visitor) override { TraceImpl(visitor); }
-
- template <typename VisitorDispatcher>
- void TraceImpl(VisitorDispatcher visitor) {
- Base::Trace(visitor);
- }
+ void Trace(Visitor* visitor) override { Base::Trace(visitor); }
};
class TraceImplBaseExtern : public Base {
public:
void Trace(Visitor* visitor) override;
- template <typename VisitorDispatcher>
- void TraceImpl(VisitorDispatcher);
-
private:
Member<X> x_;
};
« no previous file with comments | « tools/clang/blink_gc_plugin/tests/trace_after_dispatch_impl_error.txt ('k') | tools/clang/blink_gc_plugin/tests/traceimpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698