[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdeedu/kstars/kstars
From:       Akarsh Simha <akarshsimha () gmail ! com>
Date:       2008-05-29 19:28:32
Message-ID: 1212089312.105673.31528.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 814262 by asimha:

Removing redundant NULL checks for m_CurrentLink in the Details
Dialog, caused by both Jasem and me commiting the same changes.

CCMAIL: kstars-devel@kde.org



 M  +0 -11     detaildialog.cpp  


--- trunk/KDE/kdeedu/kstars/kstars/detaildialog.cpp #814261:814262
@@ -441,9 +441,6 @@
 
     if (m_CurrentLink == NULL) return;
 
-    if ( m_CurrentLink == NULL ) 
-        return;
-
     if ( m_CurrentLink->listWidget() == Links->InfoTitleList ) {
         int i = selectedObject->InfoTitle.indexOf( m_CurrentLink->text() );
         if (i >= 0) URL = QString( selectedObject->InfoList.at( i ) );
@@ -497,10 +494,6 @@
     editDialog.setButtons( KDialog::Ok | KDialog::Cancel );
     QFrame editFrame( &editDialog );
 
-    if ( m_CurrentLink == NULL ) { // We haven't selected anything
-      return;
-    }
-
     if ( m_CurrentLink->listWidget() == Links->InfoTitleList )
     {
         row = selectedObject->InfoTitle.indexOf( m_CurrentLink->text() );
@@ -596,10 +589,6 @@
 
     if (m_CurrentLink == NULL) return;
 
-    if ( m_CurrentLink == NULL ) { // We haven't selected anything
-      return;
-    }
-
     if ( m_CurrentLink->listWidget() == Links->InfoTitleList )
     {
         row = selectedObject->InfoTitle.indexOf( m_CurrentLink->text() );
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic