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:29mishok13setspambayes_score: 0.000283968 -> 0.0002839682
recipients: + mishok13, roudkerk, jnoller
2008-07-03 17:03:29mishok13setspambayes_score: 0.000283968 -> 0.000283968
messageid: <1215104608.98.0.798137013067.issue3273@psf.upfronthosting.co.za>
2008-07-03 17:03:28mishok13linkissue3273 messages
2008-07-03 17:03:27mishok13create