This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author mbussonn
Recipients mbussonn, rhettinger
Date 2018-06-21.09:24:36
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1529573076.69.0.56676864532.issue33910@psf.upfronthosting.co.za>
In-reply-to
Content
Thanks @rhettinger, this is a perfectly valid position, and one of the respons I was expecting.

>  That said, it doesn't seem to have ever caused a problem in practice (this API has been around for a very long time)

I have myself not found a lot of usage of `random.Random(x=123)` though I can't say what the lack of occurences is due to. If I have the choice between
  - Random(123)
  - Random(x=123)
  - Random(seed=123)

I definitively find `Random(x=123)` the  least readable. I can guess what `Random(123)` means, `Random(seed=123)` is self-explanatory, `Random(x=123)` will make me doubt what this means. So you might say no one as reported problem because no one is using it...

Yes `x` is explained in the docstring, but that's still requires reading the docstring and reaching for it. As some tab completion engine show functions signatures I want to argue that good parameter names could thus make a difference.

Thanks again for you quick response I'm going to close corresponding PRs.
History
Date User Action Args
2018-06-21 09:24:36mbussonnsetrecipients: + mbussonn, rhettinger
2018-06-21 09:24:36mbussonnsetmessageid: <1529573076.69.0.56676864532.issue33910@psf.upfronthosting.co.za>
2018-06-21 09:24:36mbussonnlinkissue33910 messages
2018-06-21 09:24:36mbussonncreate