d3dx8 [2/2]: add basic functions for MatrixStack

Stefan Dösinger stefandoesinger at gmx.at
Wed Dec 5 13:50:59 CST 2007


Am Mittwoch, 5. Dezember 2007 18:13:30 schrieb David.Adam at math.cnrs.fr:
> This is my first attempt with ICOM objects. So, all the advices are
> welcome.
A few suggestions:

+HRESULT WINAPI ID3DXMatrixStackImpl_QueryInterface(ID3DXMatrixStack *iface, 
REFIID riid, void **ppvObject)
+ ...
+     IClassFactory_AddRef(iface);
It is better to use ID3DXMatrixStack_AddRef(iface) here. It is essentially the 
same, but it looks more consistent

The AddRef() and Release() methods should have a TRACE which write either the 
new or old refcount, otherwise tracking refcounting bugs is hell.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20071205/ae34fe2e/attachment.pgp 


More information about the wine-patches mailing list