@Conan-Kudo requested changes on this pull request.


> @@ -97,7 +98,7 @@ dbDetectBackend(rpmdb rdb)
                rpmlog(RPMLOG_WARNING,
                        _("Converting database from %s to %s backend\n"),
                        ondisk->name, cfg->name);
-           } else {
+           } else if (*db_backend) {

Does it make sense here to have no final `else` case? This seems not right...

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1451#pullrequestreview-542832541
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to