Re: [PATCH] Pass createUser and createGroup an arguments dict.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 04, 2011 at 11:40:42AM -0500, Chris Lumens wrote:
> This makes the calls to these two functions a whole lot cleaner - instead
> of passing an enormous list of arguments, I can just convert the pykickstart
> objects into a dict and pass those in as the kwargs.
> 
> This sets up for passing more arguments in the future (for stuff like password
> controls) without having to make the method calls even larger.
> ---
>  anaconda            |    6 ++--
>  pyanaconda/users.py |   71 +++++++++++++++++++++++----------------------------
>  2 files changed, 35 insertions(+), 42 deletions(-)

This makes the call cleaner, but at the same time makes it harder for
callers to know what they can and need to pass. I think a docstring with
supported arguments would be helpful.
-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)

Attachment: pgpyZgItHA7fE.pgp
Description: PGP signature

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux