libpriv/origin: Use if-statement rather than ternary operator

More verbose, but easier to follow. Prep for adding more conditionals in that area.
coreos · Oct 14, 2021 · 77636fc · 77636fc
1 parent 7c2eef6
commit 77636fc
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/src/libpriv/rpmostree-origin.cxx b/src/libpriv/rpmostree-origin.cxx
@@ -776,10 +776,16 @@ rpmostree_origin_add_packages (RpmOstreeOrigin   *origin,
     }
 
   if (changed)
-    update_keyfile_pkgs_from_cache (origin, "packages",
-                                    local ? "requested-local" : "requested",
-                                    local ? origin->cached_local_packages
-                                          : origin->cached_packages, local);
+    {
+      const char *key = "requested";
+      GHashTable *ht = origin->cached_packages;
+      if (local)
+        {
+          key = "requested-local";
+          ht = origin->cached_local_packages;
+        }
+      update_keyfile_pkgs_from_cache (origin, "packages", key, ht, local);
+    }
 
   set_changed (out_changed, changed);
   return TRUE;