Steve Dower added the comment:
It's actually bad code generation for the switch statement in
build_filter_spec() in _lzmamodule.c.
I've filed a bug, so hopefully it will be fixed, but if not then it should be
easy enough to exclude that function (or even the whole module - _lzmamodule.c
Changes by ingrid h...@ingridcheung.com:
--
components: Tests
files: TurtleScreenBase_tests.patch
keywords: patch
nosy: ingrid, jesstess
priority: normal
severity: normal
status: open
title: Add tests for turtle.TurtleScreenBase
type: enhancement
versions: Python 3.5
Added file:
Tal Einat added the comment:
ParenMatch is indeed failing when the cursor is after the first parenthesis of
the following code:
(3 +
4 - 1)
This happens both in Shell and Editor windows.
I've traced the problem down to HyperParser. It doesn't properly support
multi-line statements, as can
Aymeric Augustin added the comment:
The idea was to not take away from what's there already: The sqlite3 module
already has a feature to inspect a command and begin or commit automatically.
Just stripping that away *removes* a feature that has been available for a
long time. I'd rather
New submission from Berker Peksag:
==
ERROR: test.test_importlib.test_locks (unittest.loader.ModuleImportFailure)
--
Traceback (most recent call last):
File
Roundup Robot added the comment:
New changeset 277d099a134b by Vinay Sajip in branch '3.4':
Issue #21752: Documented change to behaviour of logging.getLevelName().
http://hg.python.org/cpython/rev/277d099a134b
New changeset 174c30103b57 by Vinay Sajip in branch 'default':
Closes #21752: Merged
Tal Einat added the comment:
Progress: As a hack for exploring this issue, I fixed this in the Shell window
by having ParenMatch instantiate HyperParser in such a way that it parses the
entirety of the current input. In ParenMatch.flash_paren_event(), I added:
from idlelib.PyShell import
Serhiy Storchaka added the comment:
Actually iglob() can sort (per directory). But I don't think this is too needed
feature. In any case you can sort result of glob().
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
Changes by Serhiy Storchaka storch...@gmail.com:
--
resolution: - duplicate
status: open - closed
superseder: - Expand zipimport to include other compression methods
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21751
Serhiy Storchaka added the comment:
As shown in msg180323, using lzma compression for typical *.py and *.pyc
produces 8% less zip file, but reading from it is 2.5 times slower. The bzip2
compression is even worse. So there is no large benefit in supporting other
compression methods.
Vladimir Iofik added the comment:
Thanks, Martin. I didn't read the final with enough care. I think the second
part of changes (the ones that are around QueryValueEx) should be rolled back.
I believe the code was correct at that part.
Tim, what do you think?
--
Vinay Sajip added the comment:
There *is* a race condition with WatchedFileHandler - see #14632 - but there is
not much that can be done about it (see the various comments in that issue).
BTW, I wasn't able to reproduce the threading problem from your script: there
were no errors and the file
Changes by Martin Panter vadmium...@gmail.com:
--
nosy: +vadmium
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13322
___
___
Roundup Robot added the comment:
New changeset bb8b0c7fefd0 by Vinay Sajip in branch '2.7':
Issue #21742: Set stream to None after closing.
http://hg.python.org/cpython/rev/bb8b0c7fefd0
New changeset 6f1f38775991 by Vinay Sajip in branch '3.4':
Issue #21742: Set stream to None after closing.
Changes by Tal Einat talei...@gmail.com:
--
type: - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21756
___
___
Python-bugs-list mailing
.
--
components: IDLE
files: taleinat.20140614.IDLE_parenmatch_multiline_statement.patch
keywords: patch
messages: 220542
nosy: taleinat, terry.reedy
priority: normal
severity: normal
status: open
title: IDLE - ParenMatch fails to find closing paren of multi-line statements
versions: Python
Tal Einat added the comment:
I've opened a separate issue for the issue raised by Terry, #21756. Patch is
included there.
--
Added file:
http://bugs.python.org/file35627/taleinat.20140614.IDLE_parenmatch_multiline_statement.patch
___
Python tracker
Changes by Tal Einat talei...@gmail.com:
Removed file:
http://bugs.python.org/file35627/taleinat.20140614.IDLE_parenmatch_multiline_statement.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21694
Claudiu Popa added the comment:
Hi. This was already fixed in c2f1bb56760d. I'm sorry that this patch didn't
make it through.
--
nosy: +Claudiu.Popa
resolution: - fixed
stage: needs patch - resolved
status: open - closed
___
Python tracker
Stefan Krah added the comment:
Isn't PyLong_FromUnsignedLongLong() still involved through spec_add_field()?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15993
___
Giampaolo Rodola' added the comment:
It might still be worth having a time elapsed decorator
This is exactly what I think. It's a very common task.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19495
Stefan van der Walt added the comment:
This can be quite painful to work around, since the warning registry is
scattered all over. See, e.g.,
https://github.com/scikit-image/scikit-image/blob/master/skimage/_shared/_warnings.py#L9
--
nosy: +stefanv
Tal Einat added the comment:
Here are details how to trigger all of the uncovered code lines and branches,
listed by line.
Uncovered lines:
Line 159: This catches would-be identifiers (variable names) which are keywords
or begin with a character that can't be first (e.g. a digit). We should
Tal Einat added the comment:
Regarding lines 32 42, a test could also set editwin.num_context_lines to
allow triggering with smaller code blocks. It's value needs to be a list of
integers, the last of which is incredibly large, perhaps sys.maxint.
--
Claudiu Popa added the comment:
This was fixed in the latest versions.
b = bytearray(b'457')
b[:1] = 4
Traceback (most recent call last):
File stdin, line 1, in module
TypeError: can assign only bytes, buffers, or iterables of ints in range(0, 256)
--
nosy: +Claudiu.Popa
Martin v. Löwis added the comment:
Please don't. If the compiler is demonstrated to generate bad code in one case,
we should *not* exclude that code from optimization, but not use optimization
at all. How many other places will there be which also cause bad code being
generated that just
Ryan McCampbell added the comment:
If this is decided against, a partial solution would be to set the default
attribute of VAR_POSITIONAL and VAR_KEYWORD args to an empty tuple/dict,
respectively. Then you could get a parameter's value no matter what with
boundargs.get(param.name,
New submission from Mark Bell:
The following example is a Tkinter app with a button that toggles the menu
being enabled based off of
https://mail.python.org/pipermail/tkinter-discuss/2004-September/000204.html
#---
from Tkinter import *
root=Tk()
def
R. David Murray added the comment:
Given the description it sounds likely that this is a tk bug.
--
assignee: - ronaldoussoren
components: +Macintosh
nosy: +r.david.murray, ronaldoussoren
___
Python tracker rep...@bugs.python.org
Changes by R. David Murray rdmur...@bitdance.com:
--
nosy: +ned.deily
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21757
___
___
Python-bugs-list
R. David Murray added the comment:
My take is to avoid the problem entirely, and not inflict it to new users, by
providing an option to start in autocommit mode and then create transactions
only when you want them.
If this statement is accurate, the what you are proposing is just a
Changes by Saimadhav Heblikar saimadhavhebli...@gmail.com:
Added file: http://bugs.python.org/file35629/test-hyperparser-v1.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21694
___
Changes by Saimadhav Heblikar saimadhavhebli...@gmail.com:
Removed file: http://bugs.python.org/file35629/test-hyperparser-v1.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21694
___
Changes by Saimadhav Heblikar saimadhavhebli...@gmail.com:
Added file: http://bugs.python.org/file35630/test-hyperparser-v1.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21686
___
Berker Peksag added the comment:
Updated patch attached based on feedback from David. Thanks!
--
stage: needs patch - patch review
Added file: http://bugs.python.org/file35631/issue21650_v2.diff
___
Python tracker rep...@bugs.python.org
Steve Dower added the comment:
Isn't PyLong_FromUnsignedLongLong() still involved through spec_add_field()?
The two issues were unrelated - the 'invalid filter ID' (4611686018427387905 ==
0x4000_0001) is the correct value but the wrong branch in the switch
was taken, leading to the
Roundup Robot added the comment:
New changeset 42a645d74e9d by Giampaolo Rodola' in branch 'default':
fix issue #6916: undocument deprecated asynchat.fifo class.q
http://hg.python.org/cpython/rev/42a645d74e9d
--
nosy: +python-dev
___
Python tracker
Giampaolo Rodola' added the comment:
I simply removed asynchat.fifo documentation. Closing this out.
--
resolution: - fixed
status: open - closed
versions: +Python 3.5 -Python 3.2
___
Python tracker rep...@bugs.python.org
Claudiu Popa added the comment:
This seems to be a reasonable fix. Michael, could you have a look at this
patch, please?
--
nosy: +Claudiu.Popa
versions: +Python 3.5 -Python 3.2, Python 3.3, Python 3.4
___
Python tracker rep...@bugs.python.org
Changes by Giampaolo Rodola' g.rod...@gmail.com:
--
assignee: giampaolo.rodola -
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12317
___
___
Giampaolo Rodola' added the comment:
asyncore module has been deprecated as per
https://docs.python.org/3/library/asyncore.html:
This module exists for backwards compatibility only. For new code we
recommend using asyncio.
Closing this out as won't fix.
--
resolution: - wont fix
Giampaolo Rodola' added the comment:
Looking back at this: a warning seems a bit too invasive to me as there are
cases where avoiding to override a certain methods are legitimate.
I will just close this out as won't fix, also because asyncore has been
deprecated by asyncio.
--
Ben Hoyt added the comment:
Uploading a (hopefully final! :-) patch to fix Zach Ware's points from the code
review:
1) use stat.FILE_ATTRIBUTE_DIRECTORY constant in test_os.py
2) break line length in stat.rst doc source
--
Added file:
Berker Peksag added the comment:
Here's a patch.
--
keywords: +patch
nosy: +berker.peksag
stage: needs patch - patch review
Added file: http://bugs.python.org/file35633/issue18108.diff
___
Python tracker rep...@bugs.python.org
Claudiu Popa added the comment:
Looks good to me. But aren't the last two lines skipped if an error is raised
by the first line from the with block?
+with self.assertRaises(FileNotFoundError):
+shutil.chown('invalid-file', user=uid, dir_fd=dirfd)
+
Aymeric Augustin added the comment:
If this statement is accurate, the what you are proposing is just a different
(presumably clearer) spelling for 'isolation_level = None'?
This statement is accurate but it doesn't cover the whole scope of what I'm
attempting to fix.
I'm also trying to
New submission from Vajrasky Kok:
subprocess_shell in asyncio accepts cmd as a string or a bytes but the test
unit, the documentation and the exception indicates that it only accepts a
string.
--
assignee: docs@python
components: Documentation, asyncio
files:
ctismer added the comment:
Ok, I almost forgot about this because I thought my idea
was not considered, and wondered if I should keep that code online.
It is still around, and I could put it into an experimental branch
if someone asks for it.
Missing pull-request on python.org.
--
Donald Stufft added the comment:
I disagree that there is no large benefit. Python files aren't the only files
that could exist inside of a zip file. Supporting LZMA import (or bz2) would
make it easier to have LZMA or bzip2 wheels in the future without losing the
ability to import them.
Tal Einat added the comment:
With very minor changes (just use default=72 instead of or 72 and added a
comment explaining why 72 is the default - see attached), I ran the entire test
suite (which is a must according to the dev guide). On current default I'm
getting consistent failures in
Ned Deily added the comment:
Yes, it is a Tk bug. The Cocoa version of Tk 8.5 that Apple has been shipping
since OS X 10.6 has numerous problems that have been fixed in more recent
versions of Tk 8.5. With the current ActiveTcl 8.5.15, your test appears to
work correctly. Unfortunately,
Serhiy Storchaka added the comment:
For non-Python files you don't need support of bzip2 or lzma compression in
zipimport. Use zipfile which supports advanced compression.
--
___
Python tracker rep...@bugs.python.org
Donald Stufft added the comment:
I'm not sure what that statement means. There is package data that sits
alongside python files. These cannot use anything but DEFLATED because
zipimport doesn't support it.
--
___
Python tracker
Tal Einat added the comment:
Also, this seems like a small change. Should I mention it in Misc/NEWS?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20577
___
New submission from are-you-watching-closely:
Under this question
(https://docs.python.org/2/faq/programming.html#my-program-is-too-slow-how-do-i-speed-it-up)
in the 2.7 programming FAQ, it gives a link to an anecdote that Guido van
Rossum wrote.
The URL it gives:
New submission from Eric Snow:
In the documentation for the inspect module, the module type is described with
just 2 of its potential 7 attributes. The language reference[2] and importlib
docs[3] both provide an accurate list of module attributes.
Furthermore, the description for __file__
Claudiu Popa added the comment:
I believe that returning a TypeError instead of a ValueError is better in this
situation. Technically, passing 'a' as maxlen makes that value inappropiate,
thus the use of TypeError. It will also be backward compatible. Also, your
patch needs test updates.
Changes by Eric Snow ericsnowcurren...@gmail.com:
--
nosy: +brett.cannon, ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21760
___
___
Changes by Claudiu Popa pcmantic...@gmail.com:
--
stage: needs patch - patch review
versions: +Python 3.5
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19898
___
New submission from Eric Snow:
The language reference [1] says:
Ultimately, the loader is what makes use of __file__ and/or __cached__
This implies that loaders should use __file__ and __cached__ when loading a
module. Instead loaders are meant to implement exec_module() and use the spec
Changes by Eric Snow ericsnowcurren...@gmail.com:
--
assignee: - docs@python
components: +Documentation
nosy: +docs@python
stage: - needs patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21761
Claudiu Popa added the comment:
Hi. You have some trailing whitespaces in the test file (run make patchcheck or
python ../Tools/scripts/patchcheck.py). Except that, looks good to me.
--
nosy: +Claudiu.Popa
stage: - patch review
versions: +Python 3.5 -Python 3.4
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
keywords: +easy
nosy: +ezio.melotti
stage: - needs patch
type: - enhancement
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21759
___
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
--
assignee: - rhettinger
versions: -Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19898
___
Nikolaus Rath added the comment:
Nadeem, did you get a chance to look at this?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15955
___
___
New submission from Eric Snow:
With PEP 451, Python 3.4 introduced module specs to encapsulate the module's
import-related information, particularly for loading. While __loader__,
__file__, and __cached__ are no longer used by the import machinery, in a few
places it still uses __name__,
Claudiu Popa added the comment:
Hi, I left a couple of comments on rietveld.
--
nosy: +Claudiu.Popa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20069
___
Terry J. Reedy added the comment:
The current behavior is documented, if not exactly 'chosen' as the best
possible. return the indices of the opening bracket and the closing bracket
(or the end of line, whichever comes first.
As you note, the automatic matching when pausing afte typing a
Nikolaus Rath added the comment:
Raymond, it would be nice if you could respond to my last comment...
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19414
___
Tal Einat added the comment:
Note that the patch changes the behavior only for
ParenMatch.flash_paren_event(). Other uses, such as in CallTips, are not
affected.
The only possibly unwanted effect that I can think of is in an editor window,
on a line missing a closing parenthesis, triggering
Eric Snow added the comment:
__file__ is the filename from which the module *was* loaded (the inspect doc
[1] should probably reflect that [2]). The import machinery only uses the
module's __spec__ (origin, etc.). __file__ is set strictly as informational
(and for backward-compatibility).
Eric Snow added the comment:
addLevelName.__code__.co_filename
Isn't __code__ implementation-specific?
--
nosy: +eric.snow
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21709
___
New submission from Nikolaus Rath:
It is currently not perfectly clear what Python (and the standard library)
assumes about file-like objects (see e.g.
http://article.gmane.org/gmane.comp.python.devel/148199).
The attached doc patch tries to improve the current situation by stating
Eric Snow added the comment:
related: issue #17630 and issue #5950
--
nosy: +eric.snow
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17004
___
___
Eric Snow added the comment:
related: issue #17630
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21751
___
___
Python-bugs-list mailing list
New submission from Nikolaus Rath:
CPython's io.IOBase.__del__ calls self.close(), but this isn't documented
anywhere (and may be surprised for derived classes).
The attached patch extends the documentation.
--
assignee: docs@python
components: Documentation
files: iobase2.diff
Changes by Nikolaus Rath nikol...@rath.org:
Removed file: http://bugs.python.org/file35637/iobase2.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21764
___
Changes by Nikolaus Rath nikol...@rath.org:
Added file: http://bugs.python.org/file35638/iobase2.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21764
___
Changes by Mike Short bmsh...@gmail.com:
--
keywords: +patch
Added file: http://bugs.python.org/file35639/programmingfaq.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21759
___
Terry J. Reedy added the comment:
I suspect that the new end_at_eol parameter should take care of calltips.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21756
___
New submission from Terry J. Reedy:
idlelib.HyperParser.Hyperparser has these lines
_whitespace_chars = \t\n\\
_id_chars = string.ascii_letters + string.digits + _
_id_first_chars = string.ascii_letters + _
used in _eat_identifier() and get_expression. At least the latter two should
Changes by Terry J. Reedy tjre...@udel.edu:
--
dependencies: +IDLE - Test hyperparser
stage: - test needed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21756
___
Terry J. Reedy added the comment:
Since ParenMatch and fixes* to HyperParser depend on this issue, I would like
to do this first. For HyperParser.py, I would like to first commit a clean-up
only patch. The test_hyperparser patch should then only add the 'if __name__'
clause at the end.
I
Terry J. Reedy added the comment:
#21686 adds the test file that a new test would go in.
--
dependencies: +IDLE - Test hyperparser
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21765
___
Terry J. Reedy added the comment:
General rule: If the other failures happen without the patch, then go ahead.
This is, unfortunately, all too common. If they only happen with the patch
(extremely unlikely for this one), then try to find out why.
Yes, this change should have a NEWS item.
Nick Coghlan added the comment:
Manipulating name, package and path at runtime is fully supported, and the
module level attributes accordingly take precedence over the initial import
time spec.
There may be some test suite gaps and documentation issues around the
behaviour, but it's definitely
Nick Coghlan added the comment:
Can we just drop __file__ and set the origin for frozen modules to
something that includes the original file name?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21736
Nick Coghlan added the comment:
Another use case is more aggressively shrinking the bundled copy of pip. We
don't really care about speed of execution there (since we only run it
directly once at install time to do the bootstrapping), but we do care
about the impact on the installer size.
Raymond Hettinger added the comment:
Sorry Nikolaus, I'm happy with the code and docs as they are. In general, you
should assume that unless documented otherwise, any pure python container
(stdlib or 3rd party) has undefined behavior if you mutate while iterating
(like you should not assume
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
--
versions: +Python 3.5 -Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17941
___
Vinay Sajip added the comment:
Isn't __code__ implementation-specific?
It is, but ISTM it's worth getting a resolution for #21736 before another patch
for this issue is developed.
--
___
Python tracker rep...@bugs.python.org
Jim Jewett added the comment:
On Jun 14, 2014 4:05 AM, Aymeric Augustin
preserving the same behavior by default
That is a requirement, because of backwards compatibility.
providing more control for users who need a different behavior, for
instance people who use SQLite as an application
New submission from Benjamin Peterson:
From the security list:
The CGIHTTPServer Python module does not properly handle URL-encoded
path separators in URLs. This may enable attackers to disclose a CGI
script's source code or execute arbitrary scripts in the server's
document root.
Details
Roundup Robot added the comment:
New changeset b4bab0788768 by Benjamin Peterson in branch '2.7':
url unquote the path before checking if it refers to a CGI script (closes
#21766)
http://hg.python.org/cpython/rev/b4bab0788768
New changeset e47422855841 by Benjamin Peterson in branch '3.2':
url
Roundup Robot added the comment:
New changeset 601a08fcb507 by Benjamin Peterson in branch '3.4':
document IOBase.__del__'s behavior (closes #21764)
http://hg.python.org/cpython/rev/601a08fcb507
New changeset 4dca82f8d91d by Benjamin Peterson in branch '2.7':
document IOBase.__del__'s behavior
Eric Snow added the comment:
Thanks for clarifying. I remembered discussing it but couldn't recall the
details. Documenting the exact semantics, use cases, and difference between
spec and module attrs would be help. I'll look into updating the language
reference when I have some time.
It
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
--
assignee: - rhettinger
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19768
___
___
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
--
priority: normal - low
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19768
___
___
Raymond Hettinger added the comment:
Sorry, but I don't find this to be an improvement and don't think there is a
real problem worth fixing.
--
resolution: - rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 4a48450f2505 by Raymond Hettinger in branch 'default':
Issue 19898: Add test for dequereviter_new.
http://hg.python.org/cpython/rev/4a48450f2505
--
nosy: +python-dev
___
Python tracker
1 - 100 of 105 matches
Mail list logo