coverity#982292 Logically dead code

Change-Id: I7862ffc1ba51b1083e7cd1fe428fd9a0e134bc38

cui/source/customize/cfg.cxx

@@ -2130,8 +2130,8 @@ void SvxConfigPage::MoveEntry( bool bMoveUp )
 bool SvxConfigPage::MoveEntryData(
     SvTreeListEntry* pSourceEntry, SvTreeListEntry* pTargetEntry )
 {
-    //modified by shizhoubo for issue53677
-    if ( NULL == pSourceEntry || NULL == pTargetEntry )
+    //#i53677#
+    if (NULL == pSourceEntry || NULL == pTargetEntry)
     {
         return false;
     }
@@ -2142,18 +2142,6 @@ bool SvxConfigPage::MoveEntryData(
     SvxConfigEntry* pSourceData =
         (SvxConfigEntry*) pSourceEntry->GetUserData();
 
-    if ( pTargetEntry == NULL )
-    {
-        RemoveEntry( pEntries, pSourceData );
-        pEntries->insert(
-            pEntries->begin(), pSourceData );
-
-        GetSaveInData()->SetModified( true );
-
-        return true;
-    }
-    else
-    {
     SvxConfigEntry* pTargetData =
         (SvxConfigEntry*) pTargetEntry->GetUserData();
 
@@ -2175,7 +2163,6 @@ bool SvxConfigPage::MoveEntryData(
 
         return true;
     }
-    }
 
     return false;
 }