diff options
| author | Allen W. Smith, Ph.D <drallensmith@users.noreply.github.com> | 2017-08-12 15:37:09 (GMT) |
|---|---|---|
| committer | Antoine Pitrou <pitrou@free.fr> | 2017-08-12 15:37:09 (GMT) |
| commit | 48d9823a0ebde4dfab8bc154bb6df462fb2ee403 (patch) | |
| tree | 14fbb6daae62b2c83da9b7af1b309b8c64630ed0 /Misc/NEWS.d/next/Library | |
| parent | e664d7f89d2b9960d9049237136396e824795cac (diff) | |
| download | cpython-48d9823a0ebde4dfab8bc154bb6df462fb2ee403.zip cpython-48d9823a0ebde4dfab8bc154bb6df462fb2ee403.tar.gz cpython-48d9823a0ebde4dfab8bc154bb6df462fb2ee403.tar.bz2 | |
bpo-5001, bpo-31169: Fix two uninformative asserts in multiprocessing/managers.py (#3078)
* Make error message more informative
Replace assertions in error-reporting code with more-informative version that doesn't cause confusion over where and what the error is.
* Additional clarification + get travis to check
* Change from SystemError to TypeError
As suggested in PR comment by @pitrou, changing from SystemError; TypeError appears appropriate.
* NEWS file installation; ACKS addition (will do my best to justify it by additional work)
Diffstat (limited to 'Misc/NEWS.d/next/Library')
| -rw-r--r-- | Misc/NEWS.d/next/Library/2017-08-12-09-25-55.bpo-5001.huQi2Y.rst | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/Misc/NEWS.d/next/Library/2017-08-12-09-25-55.bpo-5001.huQi2Y.rst b/Misc/NEWS.d/next/Library/2017-08-12-09-25-55.bpo-5001.huQi2Y.rst new file mode 100644 index 0000000..f157d49 --- /dev/null +++ b/Misc/NEWS.d/next/Library/2017-08-12-09-25-55.bpo-5001.huQi2Y.rst @@ -0,0 +1,9 @@ +There are a number of uninformative asserts in the `multiprocessing` module, +as noted in issue 5001. This change fixes two of the most potentially +problematic ones, since they are in error-reporting code, in the +`multiprocessing.managers.convert_to_error` function. (It also makes more +informative a ValueError message.) The only potentially problematic change +is that the AssertionError is now a TypeError; however, this should also +help distinguish it from an AssertionError being *reported* by the +function/its caller (such as in issue 31169). - Patch by Allen W. Smith +(drallensmith on github). |