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?