On 2/5/2010 17:54, Paul Chitescu wrote:
Changelog:
        qedit: Add pins, IMemInputPin implementation and grabbing to 
SampleGrabber

This time I checked it applies. Sorry.
---
+/* Sample Grabber pin implementation */

+typedef struct _SG_Pin {
+    IPin pin;

This should be const IPinVtbl.

+    PIN_DIRECTION dir;
+    WCHAR const *name;
+    struct _SG_Impl *sg;
+    IPin *pair;

If it's a list why not SG_Pin* here?

+} SG_Pin;

---
Also I don't see why such helpers are needed:
---
static ULONG SampleGrabber_addref(SG_Impl *This)
---
Why not use common Ixxxx_AddRef() on grabber anywhere you need it?



Reply via email to