]> git.saurik.com Git - apt.git/commitdiff
use SPtr<pkgProblemResolver> to simply code
authorMichael Vogt <mvo@debian.org>
Tue, 27 Aug 2013 06:38:05 +0000 (08:38 +0200)
committerMichael Vogt <mvo@debian.org>
Tue, 27 Aug 2013 06:38:05 +0000 (08:38 +0200)
apt-private/private-install.cc

index d5052fcc0aeab7a43420b3cc0309fe4599d09d95..4e29fbbf63527d9ee2f3315ce98867917f149095 100644 (file)
@@ -595,7 +595,7 @@ bool DoInstall(CommandLine &CmdL)
    if (Cache->BrokenCount() != 0)
       BrokenFix = true;
 
    if (Cache->BrokenCount() != 0)
       BrokenFix = true;
 
-   pkgProblemResolver* Fix = NULL;
+   SPtr<pkgProblemResolver> Fix;
    if (_config->FindB("APT::Get::CallResolver", true) == true)
       Fix = new pkgProblemResolver(Cache);
 
    if (_config->FindB("APT::Get::CallResolver", true) == true)
       Fix = new pkgProblemResolver(Cache);
 
@@ -628,8 +628,6 @@ bool DoInstall(CommandLine &CmdL)
    if (_error->PendingError() == true)
    {
       helper.showVirtualPackageErrors(Cache);
    if (_error->PendingError() == true)
    {
       helper.showVirtualPackageErrors(Cache);
-      if (Fix != NULL)
-        delete Fix;
       return false;
    }
 
       return false;
    }
 
@@ -663,8 +661,6 @@ bool DoInstall(CommandLine &CmdL)
 
       if (_error->PendingError() == true)
       {
 
       if (_error->PendingError() == true)
       {
-        if (Fix != NULL)
-           delete Fix;
         return false;
       }
 
         return false;
       }
 
@@ -675,8 +671,6 @@ bool DoInstall(CommandLine &CmdL)
       {
         c1out << _("You might want to run 'apt-get -f install' to correct these:") << std::endl;
         ShowBroken(c1out,Cache,false);
       {
         c1out << _("You might want to run 'apt-get -f install' to correct these:") << std::endl;
         ShowBroken(c1out,Cache,false);
-        if (Fix != NULL)
-           delete Fix;
         return _error->Error(_("Unmet dependencies. Try 'apt-get -f install' with no packages (or specify a solution)."));
       }
 
         return _error->Error(_("Unmet dependencies. Try 'apt-get -f install' with no packages (or specify a solution)."));
       }
 
@@ -684,7 +678,6 @@ bool DoInstall(CommandLine &CmdL)
       {
         // Call the scored problem resolver
         Fix->Resolve(true);
       {
         // Call the scored problem resolver
         Fix->Resolve(true);
-        delete Fix;
       }
 
       // Now we check the state of the packages,
       }
 
       // Now we check the state of the packages,