sd/source/filter/ppt/ppt97animations.cxx |    5 -----
 1 file changed, 5 deletions(-)

New commits:
commit e4b165d729926692ae19c85b1596dfdaa74a15b9
Author:     Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Sun Oct 21 17:27:44 2018 +0200
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Sun Oct 21 20:17:14 2018 +0200

    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/62140
    Reviewed-by: Caolán McNamara <caol...@redhat.com>
    Tested-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sd/source/filter/ppt/ppt97animations.cxx 
b/sd/source/filter/ppt/ppt97animations.cxx
index f5729fcf1269..fe2f9010846e 100644
--- a/sd/source/filter/ppt/ppt97animations.cxx
+++ b/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 ------
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to