reapack

commit 749c56f48b51f12b2822391bee316cd51e5be23e
parent df4958e06eab108f11c708e9a547cdc86458d29c
Author: cfillion <cfillion@users.noreply.github.com>
Date:   Sun, 21 Aug 2016 17:14:06 -0400

about: remove useless safety check

Diffstat:
M
src/about.cpp
 | 
10
+++-------

1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/src/about.cpp b/src/about.cpp
@@ -389,14 +389,12 @@ const Package *AboutIndexDelegate::currentPackage() const
 
 void AboutIndexDelegate::findInBrowser()
 {
-  const Package *pkg = currentPackage();
-  if(!pkg)
-    return;
-
   Browser *browser = m_reapack->browsePackages();
   if(!browser)
     return;
 
+  const Package *pkg = currentPackage();
+
   string filter = "\"";
   filter += pkg->displayName(m_reapack->config()->browser.showDescs);
   filter += "\" \"";
@@ -409,9 +407,7 @@ void AboutIndexDelegate::findInBrowser()
 void AboutIndexDelegate::aboutPackage()
 {
   const Package *pkg = currentPackage();
-
-  if(pkg)
-    m_dialog->setDelegate(make_shared<AboutPackageDelegate>(pkg, m_reapack));
+  m_dialog->setDelegate(make_shared<AboutPackageDelegate>(pkg, m_reapack));
 }
 
 void AboutIndexDelegate::install()