Solo gp

include/gpu/GrProgramElement.h

Index: include/gpu/GrProgramElement.h
diff --git a/include/gpu/GrProgramElement.h b/include/gpu/GrProgramElement.h
index 8d23ba203576af057c6b22b21547ef048c3008c3..dfb19f8a9b2e1b3cc9fa39d5fbd5b6b63814768e 100644
--- a/include/gpu/GrProgramElement.h
+++ b/include/gpu/GrProgramElement.h
@@ -14,8 +14,8 @@
 class GrGpuResourceRef;
 
 /**
- * Base class for GrEffect (and future GrGeometryProcessor). GrDrawState uses this to manage
- * transitioning a GrEffect from being owned by a client to being scheduled for execution. It
+ * Base class for GrProcessor (and future GrGeometryProcessor). GrDrawState uses this to manage

[bsalomon, 2014/09/22 14:56:38]

delete the (and future ...) ?

Do you think it makes sense to keep this separate from GrProcessor? (I still do
think so).

[joshua.litt, 2014/09/22 18:14:44]

Yes definitely, at the very least lets see how the XferProcessor affects things.

[bsalomon, 2014/09/22 19:15:37]

Shouldn't it just read "Base class for GrProcessor."?

I thought it went:

                              GrFragmentProcessor
GrProgramElement->GrProcessor<
                              GrGeometryProcessor

+ * transitioning a GrProcessor from being owned by a client to being scheduled for execution. It
  * converts resources owned by the effect from being ref'ed to having pending reads/writes.
  *
  * All GrGpuResource objects owned by a GrProgramElement or derived classes (either directly or