pointers - Reason why not to have a DELETE macro for C++

Question

Are there any good reasons (except "macros are evil", maybe) NOT to use the following macros ?

#define DELETE( ptr ) 
if (ptr != NULL)      
{                     
    delete ptr;       
    ptr = NULL;       
}

#define DELETE_TABLE( ptr ) 
if (ptr != NULL)            
{                           
    delete[] ptr;           
    ptr = NULL;             
}

Answer 1

Personally I prefer the following

template< class T > void SafeDelete( T*& pVal )
{
    delete pVal;
    pVal = NULL;
}

template< class T > void SafeDeleteArray( T*& pVal )
{
    delete[] pVal;
    pVal = NULL;
}

They compile down to EXACTLY the same code in the end.

There may be some odd way you can break the #define system but, personally (And this is probably going to get me groaned ;) I don't think its much of a problem.