do some code-cleanup
authorDominik Heidler <dheidler@suse.de>
Tue, 31 May 2011 10:14:45 +0000 (12:14 +0200)
committerDominik Heidler <dheidler@suse.de>
Tue, 31 May 2011 10:14:45 +0000 (12:14 +0200)
src/SolverRequester.cc
src/info.cc
src/update.cc

index 15907b8..a5de91c 100644 (file)
@@ -400,7 +400,8 @@ bool SolverRequester::installPatch(
     if (ignore_pkgmgmt || patch->restartSuggested())
     {
       Patch::InteractiveFlags ignoreFlags = Patch::NoFlags;
-      ignoreFlags |= (Zypper::instance()->globalOpts().reboot_req_non_interactive) ? Patch::Reboot : Patch::NoFlags;
+      if (Zypper::instance()->globalOpts().reboot_req_non_interactive)
+        ignoreFlags |= Patch::Reboot;
 
       // bnc #221476
       if (_opts.skip_interactive && patch->interactiveWhenIgnoring(ignoreFlags))
index f3e1f02..f0c017e 100644 (file)
@@ -268,7 +268,8 @@ void printPatchInfo(Zypper & zypper, const ui::Selectable & s )
   cout << (patch->restartSuggested() ? _("Yes") : _("No")) << endl;
 
   Patch::InteractiveFlags ignoreFlags = Patch::NoFlags;
-  ignoreFlags |= (zypper.globalOpts().reboot_req_non_interactive) ? Patch::Reboot : Patch::NoFlags;
+  if (zypper.globalOpts().reboot_req_non_interactive)
+    ignoreFlags |= Patch::Reboot;
 
   cout << _("Interactive: ") << (patch->interactiveWhenIgnoring(ignoreFlags) ? _("Yes") : _("No")) << endl;
 
index 041c05d..ff160ed 100755 (executable)
@@ -140,7 +140,8 @@ static bool xml_list_patches (Zypper & zypper)
         cout << "restart=\"" << (patch->rebootSuggested() ? "true" : "false") << "\" ";
 
         Patch::InteractiveFlags ignoreFlags = Patch::NoFlags;
-        ignoreFlags |= (zypper.globalOpts().reboot_req_non_interactive) ? Patch::Reboot : Patch::NoFlags;
+        if (zypper.globalOpts().reboot_req_non_interactive)
+          ignoreFlags |= Patch::Reboot;
 
         cout << "interactive=\"" << (patch->interactiveWhenIgnoring(ignoreFlags) ? "true" : "false") << "\" ";
         cout << "kind=\"patch\"";