Gregory P. Smith g...@krypto.org added the comment:
Here's a patch for python 2.7. test cases pass but it could use review to see
if I missed any free()s.
--
keywords: +patch
Added file:
http://bugs.python.org/file24882/malloc-import-pathbufs-py27.001.diff
Ramchandra Apte maniandra...@gmail.com added the comment:
If issue7652 finishes. There's no point in fixing this bug.
--
nosy: +ramchandra.apte
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2486
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset b76fa310e73d by Eli Bendersky in branch 'default':
Issue #14207: the ParseError exception raised by _elementtree was made
http://hg.python.org/cpython/rev/b76fa310e73d
--
nosy: +python-dev
Changes by Eli Bendersky eli...@gmail.com:
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14207
___
Senthil Kumaran sent...@uthcode.com added the comment:
To note in the tracker, the original 2.6 code was changed in Issue 2254 and the
relevant changeset is c6c4398293bd
I find, Glenn's suggestion a possibly okay solution for PATH_INFO problem,
keeping in tact the security issue the the
Eli Bendersky eli...@gmail.com added the comment:
Éric, what stops us from closing this issue?
Given Ezio and Fred's feedback, I don't object to indexing cET in 3.3 - I guess
it can't hurt.
--
___
Python tracker rep...@bugs.python.org
Larry Hastings la...@hastings.org added the comment:
Attached is r3 of my patch incorporated Benjamin's suggestions.
I want #14325 to go in first.
Why do you care? If I get signoff before 14325 does please explain to me why I
should wait. They don't touch the same lines; should be a clean
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 84e4d76bd146 by Eli Bendersky in branch '3.2':
Issue #9257: clarify the events iterparse accepts
http://hg.python.org/cpython/rev/84e4d76bd146
New changeset 00c7142ee54a by Eli Bendersky in branch 'default':
Issue
Changes by Eli Bendersky eli...@gmail.com:
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9257
___
Changes by Eli Bendersky eli...@gmail.com:
--
nosy: -eli.bendersky
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12908
___
___
Python-bugs-list
Larry Hastings la...@hastings.org added the comment:
Updated patch incorporating only one of Antoine's latest suggestions. Please
see my reply on Reitveld for r3 as to why I skipped two of your three
suggestions.
--
Added file:
Senthil Kumaran sent...@uthcode.com added the comment:
Actually no. I revert back my previous statement. Meddling with splitpath
breaks all the tests.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10484
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset d2460ff173ff by Eli Bendersky in branch 'default':
Issue #13709: some fixes to the ctypes documentation. In addition to fixing
http://hg.python.org/cpython/rev/d2460ff173ff
--
nosy: +python-dev
Eli Bendersky eli...@gmail.com added the comment:
Fixed in 3.3
I don't think this is important enough to spend time on backporting.
--
resolution: - fixed
stage: needs patch - commit review
status: open - closed
versions: +Python 3.3
___
Python
Mark Dickinson dicki...@gmail.com added the comment:
No, this should be closed.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2486
___
___
Matt Joiner anacro...@gmail.com added the comment:
Is there still some value to at least exposing this in the C API, per the
precedents I mentioned?
The patch also contains some adjustment to the set_add_entry/set_add_key
abstraction dance, and some future proofing of PySet_Add return values
New submission from Matt Joiner anacro...@gmail.com:
$ python3.3 -m unittest test.test_queue
Generates errors in the unit test code of the form
AttributeError: 'BaseQueueTest' object has no attribute 'type2test'
--
components: Tests
messages: 156006
nosy: anacrolix, benjamin.peterson,
Glenn Linderman v+pyt...@g.nevcal.com added the comment:
I'm not sure what tests break or why, I suppose it is the ones for the security
patch. I'm not sure which code you are referring to... the code in msg122259 I
tested only cursorily, the (different) code in issue 13893 is what I'm
Changes by Mark Dickinson dicki...@gmail.com:
--
resolution: - rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2486
___
Michele Orrù maker...@gmail.com added the comment:
David: yes, I did. About two weeks ago.
Probably I'll take a look to those issues :)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8739
Michele Orrù maker...@gmail.com added the comment:
David, can you tag this issue as dependency for issue14261 ?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8739
___
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset bab9f29c93fd by Senthil Kumaran in branch '2.7':
2.7 - Issue #10484: Fix the CGIHTTPServer's PATH_INFO handling problem
http://hg.python.org/cpython/rev/bab9f29c93fd
New changeset 88c86869ce92 by Senthil Kumaran in
Senthil Kumaran sent...@uthcode.com added the comment:
I treated this as a regression from 2.6. A cgi url with
/cgi-bin/hello.py/hello/world would have PATH_INFO as '/hello/world'
I saw some apache specs saying that it should be 'hello/world', but I relied on
python2.6's behavior with had
Changes by Senthil Kumaran sent...@uthcode.com:
--
assignee: - orsenthil
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10487
___
___
Changes by Senthil Kumaran sent...@uthcode.com:
--
superseder: - Make CGIHTTPServer capable of redirects (and status other than
200)
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10487
___
Changes by Senthil Kumaran sent...@uthcode.com:
--
assignee: - orsenthil
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10485
___
___
Glenn Linderman v+pyt...@g.nevcal.com added the comment:
In reviewing my code in this area, I also see that in addition to fixing
_url_collapse_path_split, I override the location that uses it, which is the
is_cgi function. Here is my code for the override, which actually creates a
proper
Glenn Linderman v+pyt...@g.nevcal.com added the comment:
Senthil, the patch you submitted breaks the encapsulation of cgi_directories --
this should be changeable per particular application via subclassing or
assignment, but you've built those directories into the code. Perhaps you
should
Glenn Linderman v+pyt...@g.nevcal.com added the comment:
Another issue with the patch, is that it doesn't do .. and . collapsing on the
PATH_INFO part of the path.
It is possible for a path like
/cgi-bin/script.py/../../plain-file.html
to be passed to the server. I guess the question is if
Glenn Linderman v+pyt...@g.nevcal.com added the comment:
I just tested what Apache does with such a path as
/cgi-bin/script.py/../../plain-file.html, and it serves the plain-file.html.
--
___
Python tracker rep...@bugs.python.org
Changes by Florian Mladitsch florian.mladit...@googlemail.com:
--
keywords: +patch
Added file: http://bugs.python.org/file24885/xml.dom.pulldom.rst.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14202
Nick Coghlan ncogh...@gmail.com added the comment:
With the decision on whether or not to suppress the context moved out to a
separate flag, I think we need to allow it. Requiring that the flag be False
*and* that the context also be None gets us back to asking the question of why
the flag
New submission from Stefan Krah stefan-use...@bytereef.org:
Hi -- I'm getting a segfault running the attached crasher.py script.
Valgrind traces it down to an Invalid free() / delete / delete[] in
_PyUnicode_Ready().
Reproduce:
==
Rev: 870c0ef7e8a2
Build: ./configure --without-pymalloc
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 172630a3e6d8 by Eli Bendersky in branch '3.2':
Issue #14202: Greatly enhance the documentation of xml.dom.pulldom.
http://hg.python.org/cpython/rev/172630a3e6d8
New changeset 5d118a154ba3 by Eli Bendersky in branch
Eli Bendersky eli...@gmail.com added the comment:
I committed the patch after some minor modifications, to 3.2 and 3.3
Florian, thanks for the contribution!
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
versions: +Python 3.2 -Python 2.7
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 9fc456ac20cf by Jean-Paul Calderone in branch 'default':
Issue #14325: Stop using python lists, capsules, and the garbage collector to
deal with PyArg_Parse* cleanup.
http://hg.python.org/cpython/rev/9fc456ac20cf
Changes by Jean-Paul Calderone invalid@example.invalid:
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14325
___
Eli Bendersky eli...@gmail.com added the comment:
Éric, what else would you like to do here?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11379
___
Stefan Krah stefan-use...@bytereef.org added the comment:
3.2 also crashes. 2.7 runs fine. So it's certainly not related
to the new Unicode API.
--
versions: +Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14334
Eli Bendersky eli...@gmail.com added the comment:
This is cool! Any benchmark / speedup data?
--
nosy: +eli.bendersky
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14325
___
Jean-Paul Calderone invalid@example.invalid added the comment:
If it's a bit faster, that'd be a nice win, but I didn't benchmark. I'm
primarily interested in correctness in the PyPy case (PyPy re-uses this code),
and I think CPython benefits from the slightly simplified code as well.
If you
Eli Bendersky eli...@gmail.com added the comment:
The TreeBuilder constructor in _elemettree doesn't accept arguments, contrary
to the documentation. Added a (currently marked as failing) test.
--
___
Python tracker rep...@bugs.python.org
Stefan Krah stefan-use...@bytereef.org added the comment:
I've traced it down to this line:
getattr(type, '__getattribute__')(type, type)
Segmentation fault
Setting to 'normal', since there are apparently more of these.
--
priority: high - normal
title: Invalid free in
Éric Araujo mer...@netwok.org added the comment:
I’ll soon have a revised version of my patch to address your feedback.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11379
___
Changes by Giampaolo Rodola' g.rod...@gmail.com:
--
nosy: +giampaolo.rodola
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14320
___
___
New submission from sbt shibt...@gmail.com:
The attached patch reimplements ForkingPickler using the new dispatch_table
attribute.
This allows ForkingPickler to subclass Pickler (implemented in C) instead of
_Pickler (implemented in Python).
--
components: Library (Lib)
files:
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
One-line crasher:
type.__getattribute__(type, type)
Python 2.7 does not crash reliably, but is not exempt from the bug:
type.__getattribute__(type, 1.1j)
--
nosy: +amaury.forgeotdarc
___
Éric Araujo mer...@netwok.org added the comment:
I will commit the patch, keeping the indexing in 3.3 as agreed. I just moved
to a new city, so today is paperwork day but in the evening or tomorrow I’ll do
Python work :)
--
versions: +Python 3.3
Matt Joiner anacro...@gmail.com added the comment:
This changeset has broken something. All I get is a confusing backtrace ending
with:
File /home/matt/src/python-torrent/torrent/bencode.py, line 15, in encode
yield from encode(k)
AttributeError: 'list_iterator' object has no attribute
Benjamin Peterson benja...@python.org added the comment:
2012/3/16 Matt Joiner rep...@bugs.python.org:
Matt Joiner anacro...@gmail.com added the comment:
This changeset has broken something. All I get is a confusing backtrace
ending with:
File
Benjamin Peterson benja...@python.org added the comment:
Nevermind, I think I see the problem.
2012/3/16 Benjamin Peterson rep...@bugs.python.org:
Benjamin Peterson benja...@python.org added the comment:
2012/3/16 Matt Joiner rep...@bugs.python.org:
Matt Joiner anacro...@gmail.com added
Benjamin Peterson benja...@python.org added the comment:
Can you possibly create a small example that fails?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14230
___
sbt shibt...@gmail.com added the comment:
_eintr_retry is currently unused. The attached patch removes it.
If it is retained then we should at least add a warning that it does not
recalculate timeouts.
--
keywords: +patch
Added file:
Éric Araujo mer...@netwok.org added the comment:
Thanks to you both, the new docs are greatly better!
Eli, why not backporting to 2.7 too? It has the same rules and should get the
same attention as 3.2.
BTW, when you nest class/method directives, you needn’t repeat the class name
in the
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset b7bad204b34f by Benjamin Peterson in branch '3.2':
check to make sure the attribute is a string (#14334)
http://hg.python.org/cpython/rev/b7bad204b34f
New changeset e44591015cf0 by Benjamin Peterson in branch
Mark Shannon m...@hotpy.org added the comment:
The magic number has not been updated
Patch attached.
--
Added file: http://bugs.python.org/file24889/magic.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14230
Éric Araujo mer...@netwok.org added the comment:
Thanks for the catch! A little detective work finds that grammar.rst started
as a copy of PEP 306, where the link’s target is
http://www.python.org/sf/676521 a.k.a. http://bugs.python.org/issue676521
I can fix this later if nobody does it
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset c0a6569fdad6 by Benjamin Peterson in branch 'default':
in 72556ff86828, I should have updated the magic as well as the comment (#14230)
http://hg.python.org/cpython/rev/c0a6569fdad6
--
Changes by Éric Araujo mer...@netwok.org:
--
nosy: +eric.araujo
versions: +3rd party, Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14319
___
Changes by Éric Araujo mer...@netwok.org:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14318
___
___
Python-bugs-list mailing
Changes by Éric Araujo mer...@netwok.org:
--
stage: - needs patch
versions: +3rd party -Python 2.6, Python 2.7
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14275
___
Éric Araujo mer...@netwok.org added the comment:
Thanks for the patch. Could you rewrite it using self.assert* methods instead
of asserts?
I’m not an hmac expert, so you may have to wait a bit for another core
developer to see and commit the patch.
--
nosy: +eric.araujo
title: More
Éric Araujo mer...@netwok.org added the comment:
Could you paste the exact code that fails? In 3.2+ there is a read_something
method that takes an encoding argument, so that should work for example.
--
nosy: +eric.araujo
___
Python tracker
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset d1cf6008a565 by Benjamin Peterson in branch '2.7':
check to make sure the attribute is a string (#14334)
http://hg.python.org/cpython/rev/d1cf6008a565
--
___
Python
Éric Araujo mer...@netwok.org added the comment:
+ case $$MAKEFLAGS in *s*) quiet=-s; esac; \
Should be quiet=-q. make -s → setup.py -q
--
nosy: +eric.araujo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14330
Benjamin Peterson benja...@python.org added the comment:
It doesn't crash for me anymore.
--
nosy: +benjamin.peterson
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14334
Éric Araujo mer...@netwok.org added the comment:
Unfortunately Python 2.6 only gets fixes for security bugs now, not regular
bugs. Can you reproduce the problem with 2.7 or 3.2?
--
nosy: +eric.araujo, terry.reedy
versions: -Python 2.6
___
Python
Changes by Éric Araujo mer...@netwok.org:
--
resolution: wont fix - out of date
stage: - committed/rejected
versions: +3rd party -Python 2.6
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14317
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 1a740ea4f2a1 by Eli Bendersky in branch '3.2':
Issue #14202: some additional doc fixes
http://hg.python.org/cpython/rev/1a740ea4f2a1
New changeset b2e27f21760e by Eli Bendersky in branch 'default':
Issue #14202:
Eli Bendersky eli...@gmail.com added the comment:
Éric, thanks for the comments - all fixed.
As for 2.7, it's just laziness really. Committing to both 3.2 and 3.3 is
trivial since it's a simple merge. For 2.7 it's more complicated since it's an
unrelated branch, and the code samples have to
Brett Cannon br...@python.org added the comment:
Thanks to everyone for the reviews. I'm moving house tomorrow and I suspect
Andrea wants me to take a little Python break so I might not get to the reviews
before a2, but I will definitely get to them an merged by a3.
--
Hervé Coatanhay herve.coatan...@gmail.com added the comment:
I still have the exact same error
Maybe change should be:
#if !defined(CPU_ALLOC) defined(HAVE_SCHED_SETAFFINITY)
#undef HAVE_SCHED_SETAFFINITY
#endif
it seems CPU_ALLOC first appeared in glibc 2.7
--
Pierre Lecointre pi.lecoin...@gmail.com added the comment:
Un grand merci pour cette réponse !
La substitution index par pypi a fonctionné.
Bon week-end
2012/3/15 Alexis Metaireau rep...@bugs.python.org
Alexis Metaireau ale...@notmyidea.org added the comment:
The APIs of distutils2 have
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
well, on 2.6 and 2.7 the following has weird output and crashes:
def test(obj):
try:
type(obj).__getattribute__(obj, (1,))
except AttributeError as e:
print(e)
class C:
pass
test(str)
test(C)
test(C())
Benjamin Peterson benja...@python.org added the comment:
2012/3/16 Hervé Coatanhay rep...@bugs.python.org:
Hervé Coatanhay herve.coatan...@gmail.com added the comment:
I still have the exact same error
Maybe change should be:
#if !defined(CPU_ALLOC) defined(HAVE_SCHED_SETAFFINITY)
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 17980cb07625 by Benjamin Peterson in branch 'default':
fix condition (#14296)
http://hg.python.org/cpython/rev/17980cb07625
--
___
Python tracker rep...@bugs.python.org
Changes by R. David Murray rdmur...@bitdance.com:
--
dependencies: +Update to smtpd.py to RFC 5321
stage: - needs patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14261
___
Éric Araujo mer...@netwok.org added the comment:
I understand :) I’ll do it. I value doc as much as code, it’s also delivered
faster to the users, and there are a lot of Python 2 users out there, so
porting doc fixes to 2.7 is really worth it.
--
assignee: docs@python - eric.araujo
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 3d4d52e47431 by Benjamin Peterson in branch '2.7':
check for string attribute names in old-style classes (closes #14334)
http://hg.python.org/cpython/rev/3d4d52e47431
--
stage: needs patch -
Stefan Krah stefan-use...@bytereef.org added the comment:
I'm getting a leak since this revision:
./configure --without-pymalloc CFLAGS=-O0 -g make
valgrind --db-attach=yes --suppressions=./Misc/valgrind-python.supp
--leak-check=full ./python leak.py
==32303== 16 bytes in 1 blocks are
Stefan Krah stefan-use...@bytereef.org added the comment:
Here's leak.py.
--
Added file: http://bugs.python.org/file24890/leak.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14325
___
Éric Araujo mer...@netwok.org added the comment:
eo
--
nosy: +benjamin.peterson, eric.araujo, lemburg
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12568
___
Changes by Éric Araujo mer...@netwok.org:
--
Removed message: http://bugs.python.org/msg156059
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12568
___
Benjamin Peterson benja...@python.org added the comment:
Shouldn't using vgetargs1 with '$' in the format string just be an error?
vgetargs1 doesn't know anything about keyword arguments.
--
___
Python tracker rep...@bugs.python.org
Terry J. Reedy tjre...@udel.edu added the comment:
I reproduced the observed behavior in 3.3.0a.
However, I am rather sure it is not a bug.
In any case, linejunk is not ignored. Passing 'lambda x: 1/0' causes
ZeroDivisionError, proving that it gets called.
The body of
Marc-Andre Lemburg m...@egenix.com added the comment:
STINNER Victor wrote:
STINNER Victor victor.stin...@gmail.com added the comment:
time.clock() has been in use for ages in many many scripts.
We don't want to carelessly break all those.
I don't want to remove the function, just mark
Glenn Linderman v+pyt...@g.nevcal.com added the comment:
Senthil, from you patch, I discovered where the test_httpservers.py lives, and
added
'/cgi-bin/file1.py/../../a': ('/', 'a'),
to a local copy, and it worked fine against whatever version of the server and
tests it was
Glenn Linderman v+pyt...@g.nevcal.com added the comment:
While Amaury's comment is no doubt true, shouldn't z.read raise an exception if
it encounters an unsupported compression type?
--
nosy: +v+python
___
Python tracker rep...@bugs.python.org
Weronika Patena pat...@gmail.com added the comment:
Ah, I see. True, the ndiff docstring doesn't actually explain what junk IS - I
was just engaging in wishful thinking and assuming it did the thing I wanted.
A better explanation would help.
--
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset d08f0f3ab23e by Benjamin Peterson in branch 'default':
plug memory leak (closes #14325)
http://hg.python.org/cpython/rev/d08f0f3ab23e
--
stage: - committed/rejected
status: open - closed
Terry J. Reedy tjre...@udel.edu added the comment:
The last version in the message above looks good to me. Ready to markup and
apply?
--
keywords: +patch
nosy: +terry.reedy
stage: - patch review
___
Python tracker rep...@bugs.python.org
Changes by Terry J. Reedy tjre...@udel.edu:
--
stage: needs patch - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14250
___
___
New submission from sbt shibt...@gmail.com:
When pickling a function object, if it cannot be saved as a global the C
implementation falls back to using copyreg/__reduce__/__reduce_ex__.
The comment for the changeset which added this fallback claims that it is for
compatibility with the Python
Terry J. Reedy tjre...@udel.edu added the comment:
We know that string.Formatter does not properly duplicate str.format.
#13598 is specifically about use of {}.
Search string Formatter for other related issues.
--
nosy: +terry.reedy
resolution: - duplicate
status: open - closed
New submission from Stefan Krah stefan-use...@bytereef.org:
I don't see immediately why, but since 3877bf2e323 test_builtin and
a couple of other tests leak in refcounting mode:
hg up 8a5742b7a14d
make distclean ./configure --with-pydebug make
./python -m test -uall -R :: test_builtin
[1/1]
Benjamin Peterson benja...@python.org added the comment:
Yes, see 9e7f6ddc0d76 for the fix.
--
resolution: - out of date
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14337
Gregory P. Smith g...@krypto.org added the comment:
It looks like MAXPATHLEN is 4096 on our systems. The offending code that
caused a stack overflow segfault shows over 100 Python/import.c function calls
in its backtrace.
--
___
Python tracker
Changes by Terry J. Reedy tjre...@udel.edu:
--
nosy: +giampaolo.rodola
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14269
___
___
Terry J. Reedy tjre...@udel.edu added the comment:
I think it would be sufficient to add ' if no exceptions are raised' to the
first sentence. The example that follows in the entry clarifies the
implications of cheap to try, expensive to catch.
--
keywords: +patch
nosy: +terry.reedy
R. David Murray rdmur...@bitdance.com added the comment:
Thanks for the patch.
I think this is fine.
An alternate approach would be to introduce the concept of a state (like
imaplib has), have a list of which commands are allowed in which state, and
implement the check in the command
1 - 100 of 170 matches
Mail list logo