[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