➜
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.
Created on 2016-10-19 08:03 by franciscouzo, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| test_random.patch | franciscouzo, 2016-10-19 08:03 | review | ||
| Messages (2) | |||
|---|---|---|---|
| msg278965 - (view) | Author: Roundup Robot (python-dev)  |
Date: 2016-10-19 08:11 | |
New changeset c6588a7443a4 by Victor Stinner in branch 'default': Close #28476: Reuse math.factorial() in test_random https://hg.python.org/cpython/rev/c6588a7443a4 |
|||
| msg278966 - (view) | Author: STINNER Victor (vstinner) *  |
Date: 2016-10-19 08:13 | |
Thanks for your contribution Francisco. test_random still pass. I pushed your safe and well contained patch. It don't see any good reason why test_random had its own pure (and slow) Python implementation of factorial(), it's probably because test_random.py was written before math.factorial() was added (Python 2.6). |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:38 | admin | set | github: 72662 |
| 2016-10-19 08:13:25 | vstinner | set | nosy:
+ vstinner messages: + msg278966 |
| 2016-10-19 08:11:57 | python-dev | set | status: open -> closed nosy: + python-dev messages: + msg278965 resolution: fixed stage: resolved |
| 2016-10-19 08:03:45 | franciscouzo | create | |