[Rpm-maint] [rpm-software-management/rpm] Add a rpmKeyring iterator (PR #3365)
Panu Matilainen
notifications at github.com
Thu Oct 10 06:21:09 UTC 2024
I'll note that technically, all the "business logic" here should be in the iterator class methods - a constructor, destructor and next, and the public API functions just thin wrappers around that. Of course most of rpm looks like this at the moment but that's only because it's old code being being in slow transition towards C++, not because it's a desired state of affairs. Going forward we'll need to stop introducing more code that needs transitioning.
I don't want us to get hung up on that just now so this is NOT a requirement here.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3365#issuecomment-2404135640
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3365/c2404135640 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20241009/7dd70ddf/attachment.html>
More information about the Rpm-maint
mailing list