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 2008年07月03日 17:03 by mishok13, last changed 2022年04月11日 14:56 by admin. This issue is now closed.
| Messages (2) | |||
|---|---|---|---|
| msg69208 - (view) | Author: Andrii V. Mishkovskyi (mishok13) | Date: 2008年07月03日 17:03 | |
multiprocessing uses a lot of `assert` statements all over the code. I propose to change this way to a more readable and understandable. For example: Lib/multiprocessing/managers.py, line 136: assert isinstance(authkey, bytes) From my point of view, raising an AssertionError is not enough in this case. I propose changing this one to more intuitive: if not isinstance(authkey, bytes): raise TypeError("'authkey' argument should be an instance of 'bytes'") (Well, maybe message could be more descriptive. :) ) And this goes for all of the multiprocessing code base. Should I consider writing a patch for this? |
|||
| msg80374 - (view) | Author: Jesse Noller (jnoller) * (Python committer) | Date: 2009年01月22日 19:22 | |
I am going to track this as part of issue 5001 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:56:36 | admin | set | github: 47523 |
| 2009年01月22日 19:22:11 | jnoller | set | status: open -> closed resolution: duplicate superseder: Remove assertion-based checking in multiprocessing messages: + msg80374 |
| 2009年01月22日 19:17:24 | jnoller | set | priority: low |
| 2009年01月19日 16:42:35 | jnoller | set | assignee: jnoller |
| 2008年07月03日 17:03:28 | mishok13 | create | |