tdf#120703 (PVS): redundant nullptr check

V668 There is no sense in testing the 'pEffect' pointer against null, as
     the memory was allocated using the 'new' operator. The exception will
     be generated in the case of memory allocation error.

Change-Id: I979915efb406d3173e03b4f47883e01fe0ca5d2d
Reviewed-on: https://gerrit.libreoffice.org/62140Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

sd/source/filter/ppt/ppt97animations.cxx

@@ -581,11 +581,6 @@ void Ppt97Animation::createAndSetCustomAnimationEffect( SdrObject* pObj )
 
     //1. ------ create an effect from the presets ------
     ::sd::CustomAnimationEffectPtr pEffect( new ::sd::CustomAnimationEffect( pPreset->create( GetPresetSubType() ) ) );
-    if( !pEffect.get() )
-    {
-        DBG_ASSERT(pEffect.get(),"no suitable effect found");
-        return;
-    }
 
     //2. ------ adapt the created effect ------