Vincent Legoll added the comment:
OK I'll look at it and respin with the comments in mind
--
___
Python tracker
<http://bugs.python.org/issue11302>
___
___
Vincent Legoll added the comment:
Here we are, I left the exact messages for raised exceptions as comments so
they can easily be checked in case of test failure...
Does that look OK ?
--
Added file: http://bugs.python.org/file22495/add-more-tests-for-ast_py-2.patch
Vincent Legoll added the comment:
I was testing exact matches on the strings because I think pypy wants to be as
compliant as possible with cpython, and IMHO that include exception msgs as far
as possible. I don't see any reason to be different just for the sake of it.
As a user I&#
Vincent Legoll added the comment:
* removed the comments for exc msgs
* added explanation to multiline statements
* added 2 tests to test_AST_objects()
Regarding Éric's comment, I'm starting to wonder if those Exception tests
should be added to cpython's testsuite. Maybe tha
New submission from Vincent Legoll :
Using pylint I found some unused imports in Lib.
I filtered the most obvious ones by hand to produce the attached patch.
Should I submit individual patches, one for each file, so as to ease review ?
--
components: Library (Lib)
files: unused
Vincent Legoll added the comment:
Looking through http://hg.python.org/cpython/rev/8c17e898e0e8
I see that the glob.py hunk has been applied reversed.
The changeset 68349:55bea11d892e removing it is 3 monthes old...
Looks like I missed a svn->hg migration on my side, sorry for the scre
Vincent Legoll added the comment:
This one reverts glob.py to the previous state
--
Added file: http://bugs.python.org/file22510/reremove-sys-import-from-glob.patch
___
Python tracker
<http://bugs.python.org/issue12
New submission from Vincent Legoll :
While working to improve pypy's compliance with cpython2.7
I added more tests to the ast module test suite.
They may be of interest here too...
--
components: Tests
files: add-more-tests-for-ast_py.patch
keywords: patch
messages: 129225
Vincent Legoll <[EMAIL PROTECTED]> added the comment:
Hello,
I was searching for a bug in subprocess
module when I saw your patch.
I was implementing the exact same functionality
and mixed some of your ideas in what I use now,
which is attached...
Feel free to use it
-
Vincent Legoll <[EMAIL PROTECTED]> added the comment:
Here's a clean version with doc & test
enjoy !
Added file: http://bugs.python.org/file11263/pipeline.py
___
Python tracker <[EMAIL PROTECTED]>
<http://b
New submission from Vincent Legoll <[EMAIL PROTECTED]>:
The subprocess.Popen() object documentation should indicate
that the stdout attribute should not be modified after
object construction. Because that won't work.
Or the attribute may be rendered read-only
>>> from sub
Vincent Legoll <[EMAIL PROTECTED]> added the comment:
- Added "shut pylint up" comment for ** keyword expansion
- Added Copyright & license header
Added file: http://bugs.python.org/file11330/pipeline.py
___
Python tracker <[
New submission from Vincent Legoll :
The isgenerator() function looks duplicated, remove the one with the
shortest docstring
--
components: Library (Lib)
files: py3k-inspect.py-remove-duplicated-func.patch
keywords: patch
messages: 91172
nosy: vincele
severity: normal
status: open
title
New submission from Vincent Legoll :
The last_interval variable could potentially be used before being first
assigned a value.
--
components: Library (Lib)
files: py3k-locale.py-use-before-assignment.patch
keywords: patch
messages: 91173
nosy: vincele
severity: normal
status: open
title
New submission from Vincent Legoll :
The binhex module still has uses of the deprecated and now now removed
Carbon module.
The attached patch 'fix' this by removing the code, something else may
be required, but I don't know what.
So this is only a RFC to start discussion
--
New submission from Vincent Legoll :
The poplib modules use the 'secret' variable during its creation, this
may be a mistake. Perhaps the intention was to use the 'password' instead...
--
components: Library (Lib)
files: py3k-poplib.py-use-wrong-variable.patch
keyw
Changes by Vincent Legoll :
--
components: Library (Lib)
nosy: vincele
severity: normal
status: open
title: Lib/ftplib.py
versions: Python 3.2
___
Python tracker
<http://bugs.python.org/issue6
New submission from Vincent Legoll :
The 'macro_lines' list should probably be emptied upon leaving macro
parsing mode.
Simplify code by using the 'macro_name' variable as the boolean for
macro parsing mode.
Deprecated code probably should be fixed until completely removed
Vincent Legoll added the comment:
Yes it does
--
___
Python tracker
<http://bugs.python.org/issue6620>
___
___
Python-bugs-list mailing list
Unsubscribe:
Vincent Legoll added the comment:
Could someone ensure the attached unittest does the job, I blindly
copied from above...
I don't know how to launch an unittest with the svn 'Lib', it takes the
system one and just messing with PYTHONPATH won't work either.
Vincent Legoll added the comment:
If the macro_lines is not emptied upon end of parsing a macro
definition, if there's another macro definiton it will 'inherit' the
lines from the previous one, which could very well be uncool.
The use before definition should also be fixed.
Ple
New submission from Vincent Legoll :
As per the blog entry
http://www.logilab.org/blogentry/17873
I think the tempfile.mkstemp() documentation could be more helpful by
suggesting the use of os.close() appropriately.
If some native english speaker could give a review of the language I
used in
Vincent Legoll added the comment:
Or a review of the markup I used
--
___
Python tracker
<http://bugs.python.org/issue6875>
___
___
Python-bugs-list mailin
Vincent Legoll added the comment:
The real question I had is why mkstemp return an os-level opened file
descriptor instead of a python file object...
--
___
Python tracker
<http://bugs.python.org/issue6
New submission from Vincent Legoll:
Here is a patch to fix a missing word in the urllib doc
--
assignee: docs@python
components: Documentation
files: python-doc-fix.hgpatch
messages: 245927
nosy: docs@python, vincent-legoll
priority: normal
severity: normal
status: open
title: [doc
Changes by Vincent Legoll :
--
nosy: +vincent-legoll
___
Python tracker
<http://bugs.python.org/issue25940>
___
___
Python-bugs-list mailing list
Unsubscribe:
Vincent Legoll added the comment:
Maybe if the server change is not approved you can still push the part of the
patch that un-hardwire that server name everywhere...
--
___
Python tracker
<http://bugs.python.org/issue25
27 matches
Mail list logo
