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 | mishok13 |
|---|---|
| Recipients | jnoller, mishok13, roudkerk |
| Date | 2008年07月03日.17:03:27 |
| SpamBayes Score | 0.0002839682 |
| Marked as misclassified | No |
| Message-id | <1215104608.98.0.798137013067.issue3273@psf.upfronthosting.co.za> |
| In-reply-to |
| Content | |
|---|---|
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? |
|
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2008年07月03日 17:03:29 | mishok13 | set | spambayes_score: 0.000283968 -> 0.0002839682 recipients: + mishok13, roudkerk, jnoller |
| 2008年07月03日 17:03:29 | mishok13 | set | spambayes_score: 0.000283968 -> 0.000283968 messageid: <1215104608.98.0.798137013067.issue3273@psf.upfronthosting.co.za> |
| 2008年07月03日 17:03:28 | mishok13 | link | issue3273 messages |
| 2008年07月03日 17:03:27 | mishok13 | create | |