[Rpm-maint] [rpm-software-management/rpm] Add support for application private user data in transaction elements (#1366)

Jaroslav Rohel notifications at github.com
Tue Sep 22 10:23:33 UTC 2020


@jrohel 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"},

`"te.SetPriv(userdata) -- Set associated user data (if any)\n"`

Note:
I prefer `userdata` instead of `priv`.
Example: `SetUserData ()`, `GetUserData ()`/`UserData ()`. Below the `priv`, I can imagine a something other.
And in the callback `void *userdata` instead of` void *priv`.
But that's my feeling.


-- 
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#pullrequestreview-493318411
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20200922/79405cc8/attachment.html>


More information about the Rpm-maint mailing list