@pmatilai commented on this pull request.


> @@ -35,7 +36,8 @@ public:
     rpmRC delete_key(rpmtxn txn, rpmPubkey key) override;
     rpmRC delete_store(rpmtxn txn) override;
 private:
-    rpmRC delete_key(rpmtxn txn, const std::string & keyid, unsigned int 
newinstance = 0);
+    friend rpmRC delete_key(auto keystore, rpmtxn txn, rpmPubkey key, auto 
skip);

Lets group these friend declarations separate from the actual methods for 
clarity.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/4005#pullrequestreview-3349328879
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/4005/review/[email protected]>
_______________________________________________
Rpm-maint mailing list
[email protected]
https://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to