[Rpm-maint] [rpm-software-management/rpm] Add support for application private user data in transaction elements (#1366)
Panu Matilainen
notifications at github.com
Mon Sep 28 07:56:31 UTC 2020
@pmatilai commented on this pull request.
> as passed e.g. as data arg ts.addInstall()"},
+ {"Priv", (PyCFunction)rpmte_Priv, METH_NOARGS,
+ "te.Priv() -- Return associated user data (if any)\n"},
+ {"SetPriv", (PyCFunction)rpmte_SetPriv, METH_O,
+ "te.Priv() -- Set associated user data (if any)\n"},
Fair enough, not sure what was supposed to be the idea behind using "priv" there. The only easy thing about names is picking a bad one :sweat_smile:
Changed to rpmte(Set)Userdata etc now, thanks for the feedback.
--
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/1366#discussion_r495753609
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20200928/23e41685/attachment.html>
More information about the Rpm-maint
mailing list