Dustin J. Mitchell added the comment:
If the patch still applies cleanly, I have no issues with you or anyone opening
a PR. I picked this up several years ago at the PyCon sprints, and don't
remember a thing about it, nor have I touched any other bit of the CPython
source since then. S
New submission from Dustin J. Mitchell:
Reproduction:
# main.py
import foo.bar
# foo/__init__.py
# (empty)
# foo/bar/__init__.py
import foo.bar.bing as bing
# foo/bar/bing.py
# (empty)
Result:
dustin@euclid ~/tmp $ python3.3 main.py
Traceback (most recent call last):
File "ma
Dustin J. Mitchell added the comment:
This seems to have stalled out after the PyCon sprints. Any chance the final
patch can be reviewed?
--
___
Python tracker
<http://bugs.python.org/issue3
Dustin J. Mitchell added the comment:
New:
- rename token symbols in token.h with a PYTOK_ prefix
- include an example of using the PyTokenizer functions
- address minor review comments
--
Added file: http://bugs.python.org/file38999/issue3353-2.patch
Dustin J. Mitchell added the comment:
Here's an updated patch for #1:
Existing Patch:
- move tokenizer.h from Parser/ to Include/
- Add PyAPI_Func to export tokenizer functions
New:
- Removed unused, undefined PyTokenizer_RestoreEncoding
- Include PyTokenizer_State with limite
Dustin J. Mitchell added the comment:
>From my read of this bug, there are two distinct tasks mentioned:
1. make PyTokenizer_* part of the Python-level API
2. re-implement 'tokenize' in terms of that Python-level API
#1 is largely complete in Andrew's latest patch, but tha
Dustin J. Mitchell added the comment:
This fixes the four cases Nick referred to, although not in the functions
expected:
- str.encode no longer exists
- unicode.decode no longer exists
- encoders used by str.encode must return bytes (does not apply to
codecs.encode)
- decoders used by
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
Thoughts on this patch?
___
Python tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1187>
___
_
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
Hmm.. I see why you didn't write a unit test for this!
Attached is a patch with a unit test that tickles this behavior, along
with Andrew's fix.
Added file: http://bugs.python.org/file10338/1
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
Ack, sorry. My 'vi' settings should now be correct.
Added file: http://bugs.python.org/file10274/1174606-3.patch
_
Tracker <[EMAIL PROTECTED]>
<http://bug
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
Thanks, Adam -- requested changes made
Added file: http://bugs.python.org/file10271/1174606-2.patch
_
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.o
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
I see that, on running your fix_fileno.py, nothing is output to
/tmp/stdout.test. I don't necessarily see the link to your fix. Could
you describe the problem and/or add comments to your patch to explain why
these chec
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
Improved fix; this passes test_file on my system.
--
keywords: +patch
Added file: http://bugs.python.org/file10266/1174606.patch
_
Tracker <[EMAIL PROTECTED]>
<http://
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
This test passes -- is this ready to commit?
I see a reduction from 1.9s to 1.5s for the test script in msg59715.
--
nosy: +djmitche
__
Tracker <[EMAIL PROTECTED]>
<http://
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
While it is a sensible fix for the signed/unsigned problem, David's patch
still fails regrtest on my system (amd64), after OOM-killing random other
processes :(
Andrew's suggestion makes a lot of sense here. Does i
Dustin J. Mitchell <[EMAIL PROTECTED]> added the comment:
Patch attached.
--
keywords: +patch
Added file: http://bugs.python.org/file10258/1625509.patch
_
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.o
16 matches
Mail list logo