Inlining SAFE_RELEASE()?

May I ask why you want it as an inline function...? I don''t think you can make it any faster than it already is...

// Javelin
-- Why do something today when you can do it tomorrow... --

Hm, to do SAFE_RELEASE as an inlined function, one has to pass a pointer to a pointer to be able to set it to NULL. Sp the function may look like this (not tested nor used):

template < class T > void SafeRelease( T** p ) { if ( *p != NULL ) { (*p)->Release(); (*p) = NULL; } }

or

void SafeRelease( IUnknown** p ) { if ( *p != NULL ) { (*p)->Release(); (*p) = NULL; } }

BTW: I agree with Javelin. There is actually no reason to do so.

Bjørn.

Edited by - Boki on August 7, 2001 4:34:07 PM

Well, I was under the impression that it is better to use inline, constant, and enum in favor of #define.

I suppose because I figured if I''m going to program in C++ over C, may as well go all the way, right?

-----------------
The Goblin (madgob@aol.com)
-----------------
"Before critisizing somebody, walk a mile in their shoes. That way, when you do critisize them, not only will you be a mile away, but you''ll also have their shoes!"

The better way would be to use the _com_ptr_t class defined in COMDEF.H or an equivalent class.

Bjørn.