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 valhallasw
Recipients Ronny.Pfannschmidt, RonnyPfannschmidt, alexandre.vassalotti, flox, georg.brandl, ggenellina, gvanrossum, jcea, jdharper, kmike, loewis, pitrou, valhallasw
Date 2013-12-06.23:22:12
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1386372132.7.0.432169996226.issue6784@psf.upfronthosting.co.za>
In-reply-to
Content
I have fixed most of the nits in this patch, except for:

1) the intermediate bytes object being created; inlining is an option, as storchaka suggested, but I'd rather have you decide what it should become before implementing it;

2) make clinic gives me 

./python -E ./Tools/clinic/clinic.py --make
Error in file "./Modules/_pickle.c" on line 6611:
Checksum mismatch!
Expected: bed0d8bbe1c647960ccc6f997b33bf33935fa56f
Computed: 58dcccb705487695fec30980f566027bc68d9c69
make: *** [clinic] Error 255

and I have no clue how to fix that -- the clinic docs are sparse, to say the least;

3) The tests are still in their own test case; please decide between the two of you what is the best solution;

4) I have grouped the test cases: test_load_python2_str_as_bytes (which checks protocols 0, 1, and 2), test_load_python2_unicode_as_str and test_load_long_python2_str_as_bytes;

5) I have moved the commands to create the shown pickled versions from docstrings to comments. If you think they are not useful, I'll remove them, but I found them pretty useful while shortening the strings.
History
Date User Action Args
2013-12-06 23:22:12valhallaswsetrecipients: + valhallasw, gvanrossum, loewis, georg.brandl, jcea, ggenellina, pitrou, alexandre.vassalotti, RonnyPfannschmidt, flox, jdharper, Ronny.Pfannschmidt, kmike
2013-12-06 23:22:12valhallaswsetmessageid: <1386372132.7.0.432169996226.issue6784@psf.upfronthosting.co.za>
2013-12-06 23:22:12valhallaswlinkissue6784 messages
2013-12-06 23:22:12valhallaswcreate