[issue17468] Generator memory leak

Nick Coghlan added the comment: Issue 17800 is anyone wants to weigh in on the tp_del -> __del__ question (I ended up not adding Guido back to that one either, since the original design was based on an assumption that's now demonstrably false, so it makes sense to update the behaviour) --

[issue17800] Expose __del__ when tp_del is populated from C code

New submission from Nick Coghlan: This came up in issue 17468: currently, populating tp_del from C (as generators now do) doesn't automatically create a __del__ wrapper visible from Python. The rationale given in the initial commit is that there's no need to define a wrapper, since tp_del won'

[issue17468] Generator memory leak

Nick Coghlan added the comment: I'll create a separate issue for the tp_del -> __del__ question, since that's a language design decision that *does* need Guido's input, but doesn't relate to the broader question of generators, cycles and potential memory leaks. --

[issue3693] Obscure array.array error message

Alexandre Vassalotti added the comment: Here's a patch to fix the exception. -- keywords: +patch Added file: http://bugs.python.org/file29949/fix_array_err_msg.patch ___ Python tracker __

[issue17799] settrace docs are wrong about "c_call" events

New submission from Ned Batchelder: Looking into this Stack Overflow question: http://stackoverflow.com/questions/16115027/pythons-sys-settrace-wont-create-c-call-events Reading the code in c_eval.c and friends, it looks like "c_call" events are never passed to the trace function, only to the

[issue17468] Generator memory leak

Changes by Benjamin Peterson : -- nosy: -gvanrossum ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai

[issue13262] IDLE opens partially hidden

Terry J. Reedy added the comment: That does not work because Idle just moves the window down to compensate. (It seems to take two openings to have that happen.) Even if that worked, I would still ask why I should have to make the window smaller, when I want it larger (44 lines), located where

[issue7951] Should str.format allow negative indexes when used for __getitem__ access?

Todd Rovito added the comment: Here is a simple patch that simply explains negative indexes and negative slices are not supported for the string format documentation. Perhaps more documentation needs to be created else where to help explain why all collections do not need to support negative

[issue17468] Generator memory leak

Guido van Rossum added the comment: Unless I'm the only person on earth who can understand this I beg to be left out of this bug. I just don't have the bandwidth right now to look into it. If it's really about GC and __del__ and generators, maybe Tim Peters would be willing to look into it? Or

[issue17646] traceback.py has a lot of code duplication

Benjamin Peterson added the comment: It would be great to have a test for that. :) -- ___ Python tracker ___ ___ Python-bugs-list mail

[issue13262] IDLE opens partially hidden

Roger Serwy added the comment: Why not change the initial window size from 80x40 to 80x38? -- ___ Python tracker ___ ___ Python-bugs-l

[issue13262] IDLE opens partially hidden

Terry J. Reedy added the comment: For me, this is currently about the most annoying behavior of IDLE. -- nosy: +roger.serwy versions: +Python 2.7, Python 3.3 ___ Python tracker _

[issue17798] IDLE: can not edit new file names when using -e

New submission from Roger Serwy: This is a split from #13495 as it is truly a separate issue. Here's a partial quote from msg151278: """ IDLE 3 no longer allows for editing of new files from the command line. For example: idle -e /tmp/newfile.py will momentarily flash an editor window b

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

Roundup Robot added the comment: New changeset d499189e7758 by R David Murray in branch 'default': #9607: restore keywords.kwlist after testing it. http://hg.python.org/cpython/rev/d499189e7758 -- ___ Python tracker

[issue13495] IDLE: Regressions - Two ColorDelegator instances loaded

Roger Serwy added the comment: I will split the "good_load" flag bug into its own issue. While addressing a bug in #5492, another doubled-up ColorDelegator problem appeared, tracing back to #1586. The dual_delagators patch removes the code which loads the extra color delegators as well as ens

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

R. David Murray added the comment: Wow, it sure took me a long time to get back to this. Greg, the test ideas in your patch were great, and I used them all, but I wound up rewriting the actual code to better match the style of our other unit tests, and also to take advantage of some of our te

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

Roundup Robot added the comment: New changeset 0ad18bb85371 by R David Murray in branch 'default': #9607: Add tests for the keyword module. http://hg.python.org/cpython/rev/0ad18bb85371 -- nosy: +python-dev ___ Python tracker

[issue5492] Error on leaving IDLE with quit() or exit() under Linux

Roger Serwy added the comment: Good catch Terry! I've been testing using "python -m idlelib.idle" instead of importing it from an interactive prompt. I'll need to remember to consider that test vector in the future. I figured out why those messages are popping up. The Tk event loop remains ru

[issue15392] Create a unittest framework for IDLE

Tomoki Imai added the comment: I'm a student thinking of participating in Google Summer of Code. And want to work to create a unittest for IDLE. Using unittest.mock seemed to be good way to test GUI. But there is a problem. There is no unittest.mock in Python2. http://docs.python.org/2/library/u

[issue17745] "packaging" no longer planned to be included

Tshepang Lekhonkhobe added the comment: There should at least be a mention that 'packaging' is no longer planned to be included, to avoid any confusion/misinformation. -- ___ Python tracker ___

[issue17646] traceback.py has a lot of code duplication

Martin Morrison added the comment: print_tb() wasn't passing the file argument through to print_list() - now fixed. I also verified none of the other functions had a similar bug. -- Added file: http://bugs.python.org/file29944/issue17646-4.diff ___ P

[issue17646] traceback.py has a lot of code duplication

Benjamin Peterson added the comment: Also, this patch seems to break test_zipimport. -- ___ Python tracker ___ ___ Python-bugs-list ma

[issue17642] IDLE add font resizing hot keys

Changes by Todd Rovito : -- nosy: +Todd.Rovito ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyth

[issue12535] Chained tracebacks are confusing because the first traceback is minimal

Changes by Terry J. Reedy : -- type: behavior -> enhancement versions: +Python 3.4 -Python 3.3 ___ Python tracker ___ ___ Python-bugs-

[issue14805] Support display of both __cause__ and __context__

Changes by Martin Morrison : -- nosy: +isoschiz ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue12535] Chained tracebacks are confusing because the first traceback is minimal

Changes by Martin Morrison : -- nosy: +isoschiz ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue17762] platform.linux_distribution() should honor /etc/os-release

Éric Araujo added the comment: os-release finally provides a cross-OS release file with a specification. I think it should be authoritative, then the lsb-release system (it’s officially a script but many OSes just parse a static file), then OS-specific files. -- assignee: -> lemburg

[issue17761] platform._parse_release_file doesn't close the /etc/lsb-release file, doesn't know about 'Ubuntu'

Éric Araujo added the comment: +1 to with. The patch also includes an unrelated change. -- nosy: +eric.araujo ___ Python tracker ___

[issue17745] "packaging" no longer planned to be included

Éric Araujo added the comment: The role of the PEP was to announce changes; I don’t know if changing it now is useful, as the release is done and its release notes are accurate. -- assignee: docs@python -> georg.brandl components: -Distutils, Distutils2, Documentation nosy: +eric.arauj

[issue17735] inspect.findsource raises IndexError

Changes by Éric Araujo : -- title: inspect.findsource throws IndexError -> inspect.findsource raises IndexError ___ Python tracker ___ __

[issue17732] distutils.cfg Can Break venv

Éric Araujo added the comment: Looks good, thanks! I assume you also tested it manually? I’ll take care of this for the next release. -- nosy: +georg.brandl, larry priority: normal -> release blocker ___ Python tracker

[issue16741] `int()`, `float()`, etc think python strings are null-terminated

Changes by Martin Morrison : -- nosy: +isoschiz ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue11549] Build-out an AST optimizer, moving some functionality out of the peephole optimizer

Changes by Martin Morrison : -- nosy: +isoschiz ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue17646] traceback.py has a lot of code duplication

Martin Morrison added the comment: How embarrassing - I guess that'll teach me to try to doctor the diff file. :-) I've now regenerated the diff, and re-run all the testing, etc. Apologies for the churn. -- Added file: http://bugs.python.org/file29943/issue17646-3.diff ___

[issue17618] base85 encoding

Changes by Serhiy Storchaka : Added file: http://bugs.python.org/file29942/issue17618-5.diff ___ Python tracker ___ ___ Python-bugs-list maili

[issue17742] Add _PyBytesWriter API

Serhiy Storchaka added the comment: _PyBytesWriter and _PyUnicodeWriter have differen use cases. While _PyUnicodeWriter used primary in formatter where resulting size is rarely known and reallocation in decoders usually caused by widening result, _PyBytesWriter is used only in decoders where w

[issue16394] Reducing tee() memory footprint

Changes by Raymond Hettinger : -- assignee: docs@python -> rhettinger nosy: +rhettinger ___ Python tracker ___ ___ Python-bugs-list ma

[issue16863] Python 2 error in Argparse tutorial

Changes by Phil Connell : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue17306] Improve the way abstract base classes are shown in help()

Changes by Phil Connell : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue11698] Improve repr for structseq objects to show named, but unindexed fields

Changes by Phil Connell : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue11549] Build-out an AST optimizer, moving some functionality out of the peephole optimizer

Changes by Phil Connell : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue17718] boolop constant checking for if/while

Benjamin Peterson added the comment: Thank you for the patch. However, I don't think such "optimizations" are particularly useful. -- ___ Python tracker ___

[issue17718] boolop constant checking for if/while

Changes by Terry J. Reedy : -- nosy: +benjamin.peterson, brett.cannon, georg.brandl, ncoghlan ___ Python tracker ___ ___ Python-bugs-l

[issue17714] str.encode('base64') add trailing new line character. It is not documented.

Changes by Terry J. Reedy : -- assignee: -> docs@python components: +Documentation nosy: +docs@python stage: -> needs patch ___ Python tracker ___ _

[issue17705] Fill Character cannot be \0

Changes by Terry J. Reedy : -- versions: -Python 3.2 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://ma

[issue17792] Unhelpful UnboundLocalError due to del'ing of exception target

Eric Snow added the comment: FWIW this has come up before: http://mail.python.org/pipermail/python-dev/2012-October/122504.html and relatedly: issue16429 -- nosy: +eric.snow ___ Python tracker __

[issue17729] advocacy howto improvements

Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue16394] Reducing tee() memory footprint

Changes by Phil Connell : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue12489] email.errors.HeaderParseError if base64url is used

Changes by Phil Connell : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue15992] Strict aliasing violations in Objects/unicodeobject.c

Changes by Phil Connell : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue17123] Add OCSP support to ssl module

Changes by Phil Connell : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue17729] advocacy howto improvements

Tshepang Lekhonkhobe added the comment: I would rather the page be removed, at least from the official docs. I wonder how useful it actually is, and as already pointed, it really feels outdated (and Apache is not kool no more). -- ___ Python tracker

[issue17729] advocacy howto improvements

Changes by Tshepang Lekhonkhobe : -- nosy: +tshepang ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai

[issue17758] test_site fails when the user does not have a home directory

Changes by Tshepang Lekhonkhobe : -- nosy: +tshepang ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai

[issue17780] the test suite should use a TEMPDIR in the build directory, not the source directory

Changes by Tshepang Lekhonkhobe : -- nosy: +tshepang ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai

[issue17785] Use faster URL shortener for perf.py

Changes by Tshepang Lekhonkhobe : -- nosy: +tshepang ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai

[issue15721] PEP 3121, 384 Refactoring applied to tkinter module

Andrew Svetlov added the comment: No, PEP3121 patch is not ready yet but PEP384 has been applied. -- ___ Python tracker ___ ___ Python

[issue17468] Generator memory leak

Changes by Martin Morrison : -- nosy: +pconnell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue17778] Fix test discovery for test_multiprocessing.py

Zachary Ware added the comment: Hmmm, it looks a couple of things have gone wrong. First, I didn't try the patch on 3.3, whose test_multiprocessing is significantly different than default's. This appears to be due largely to dd5e98ddcd39, but there have been other changes to only default sin

[issue17413] format_exception() breaks on exception tuples from trace function

R. David Murray added the comment: Benjamin reviewed the patch and pointed out that the settrace state needed to be restored in the test, so I fixed that when I committed it. Thanks Ingrid and Brendan. -- resolution: -> fixed stage: commit review -> committed/rejected status: open ->

[issue17413] format_exception() breaks on exception tuples from trace function

Roundup Robot added the comment: New changeset d18df4c90515 by R David Murray in branch '3.3': #17413: make sure settrace funcs get passed exception instances for 'value'. http://hg.python.org/cpython/rev/d18df4c90515 New changeset 6297fcddf912 by R David Murray in branch 'default': Merge #17413

[issue17732] distutils.cfg Can Break venv

Nick Sloan added the comment: Here is another update. It has come to my attention that I missed some options: prefix, exec-prefix, home, user and root These have been added, and the docs and test have been updated to reflect the change. -- Added file: http://bugs.python.org/file29940/

[issue7951] Should str.format allow negative indexes when used for __getitem__ access?

Changes by Todd Rovito : -- nosy: +Todd.Rovito ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pytho

[issue17468] Generator memory leak

Antoine Pitrou added the comment: > Yes. Hopefully, the async framework using generators can properly can > close() on them, though. With yield from-based generators, you don't need a trampoline anymore, so the close() call is now left to the application developers (if I'm not mistaken). --

[issue17468] Generator memory leak

Benjamin Peterson added the comment: Yes. Hopefully, the async framework using generators can properly can close() on them, though. -- ___ Python tracker ___ ___

[issue16316] Support xz compression in mimetypes module

Éric Araujo added the comment: Small changes in registries (mimetypes, html.entities, sometimes webbrowser) are acceptable in stable branches. Can you backport this? -- stage: committed/rejected -> commit review status: closed -> open ___ Python tra

[issue17468] Generator memory leak

Benjamin Peterson added the comment: I think the creation of __del__ wrappers for extension types is separate from this issue of generator finalization. -- ___ Python tracker __

[issue17796] No mimetype guessed for tar.xz url

Éric Araujo added the comment: Let’s keep using that other bug report. -- resolution: -> duplicate stage: needs patch -> committed/rejected status: open -> closed superseder: -> Support xz compression in mimetypes module ___ Python tracker

[issue17796] No mimetype guessed for tar.xz url

Éric Araujo added the comment: +1 -- nosy: +eric.araujo stage: -> needs patch ___ Python tracker ___ ___ Python-bugs-list mailing lis

[issue17468] Generator memory leak

Antoine Pitrou added the comment: With the advent of yield-based asynchronous programming, it is going to be problematic if a generator caught in a reference cycle can create a memory leak. -- nosy: +pitrou ___ Python tracker

[issue17764] Support http.server passing bind address via commend line argument

Berker Peksag added the comment: I left comments on Rietveld. The patch also needs a documentation update. -- nosy: +berker.peksag stage: -> patch review ___ Python tracker ___

[issue17468] Generator memory leak

Nick Coghlan added the comment: Adding Guido because this appears to be due to a longstanding difference between the handling of tp_del and most other slots Specifically, the reason for the odd behaviour appears to be that generator objects define tp_del [1] (which is what the cyclic gc *reall

[issue17177] Document/deprecate imp

Brett Cannon added the comment: Nah, consider it motivation for me to get it done in Python 3.4 else I have to back it out before release. -- ___ Python tracker ___

[issue3778] python uninstaller leave registry entries

Changes by Brian Curtin : -- nosy: -brian.curtin ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue3778] python uninstaller leave registry entries

Changes by Michael Foord : -- nosy: -michael.foord ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue3778] python uninstaller leave registry entries

anatoly techtonik added the comment: What is the reason for not following the good practice? Is it so hard to fix? -- ___ Python tracker ___ __

[issue17177] Document/deprecate imp

R. David Murray added the comment: Should we back out that module level deprecation notice for the moment, then? -- ___ Python tracker ___ ___

[issue17732] distutils.cfg Can Break venv

Nick Sloan added the comment: Responded to comments with an updated patch. Thanks for all the feedback, and sorry for the silly mistakes. Should have read up more thoroughly on the docs style guide and the terminology. Hopefully the latest patch is ready to go (or at least, nearly so). --

[issue17797] Visual C++ 11.0 reports fileno(stdin) == 0 for non-console program

Mateusz Loskot added the comment: Replacing if the current test in Python 3.2 if (fd < 0) with if (check_fd(fd) < 0) Seems to be a working solution. I just noticed, that in current Python/pythonrun.c in the repo, there the fd < 0 tests have been replaced with new function is_valid_fd(). But,

[issue17177] Document/deprecate imp

Brett Cannon added the comment: The deprecation warnings for the explicitly deprecated functions already have the instructions on how to port. A note could probably be added to turn on deprecation warnings if one needs help with moving code over. But the deprecation isn't even officially full-

[issue17468] Generator memory leak

Benjamin Peterson added the comment: A documentation patch sounds good to me. -- assignee: -> docs@python components: +Documentation nosy: +docs@python type: resource usage -> behavior ___ Python tracker _

[issue17468] Generator memory leak

Anssi Kääriäinen added the comment: True, except GeneratorExit will run at garbage collection time and this will result in reference cycle problems. Checking if there is no except GeneratorExit clause might be too complicated. I still think this is worth a brief note in the gc docs. The gc doc

[issue17797] Visual C++ 11.0 reports fileno(stdin) == 0 for non-console program

Amaury Forgeot d'Arc added the comment: Maybe check_fd(fd) could be used in initstdio as well. Can you check whether it's the same for the other files? What are the values for fileno(stdout) and fileno(stderr)? -- ___ Python tracker

[issue17796] No mimetype guessed for tar.xz url

R. David Murray added the comment: Adding xz was treated as an enhancement in issue 16316. Our thinking about applying these types of changes as bug fixes has evolved a bit, so I think we can consider backporting it. -- nosy: +r.david.murray ___ Py

[issue17797] Visual C++ 11.0 reports fileno(stdin) == 0 for non-console program

Mateusz Loskot added the comment: > In file Modulfileio.c, I messed the path and filename above I meant: In file Modules/_io/fileio.c, -- ___ Python tracker ___

[issue5004] socket.getfqdn() doesn't cope properly with purely DNS-based setups

R. David Murray added the comment: That is an interesting proposal, yes. I suppose someone that needs the getaddrinfo semantics for something other than the local host can just call it directly. Now, do we add the fact that we are doing this to the current alogarithmic documention? :) -

[issue17468] Generator memory leak

Benjamin Peterson added the comment: Generators exit using GeneratorExit, which you could possible catch. -- ___ Python tracker ___ __

[issue17797] Visual C++ 11.0 reports fileno(stdin) == 0 for non-console program

Mateusz Loskot added the comment: Yes, it does. In file Modulfileio.c, in function fileio_init, there is this code: if (fd >= 0) { if (check_fd(fd)) goto error; self->fd = fd; self->closefd = closefd; } The check_fd tests: if (!_PyVerify_fd(fd) || (

[issue17797] Visual C++ 11.0 reports fileno(stdin) == 0 for non-console program

Amaury Forgeot d'Arc added the comment: And does it cause an issue later? How? -- nosy: +amaury.forgeotdarc ___ Python tracker ___ ___

[issue17797] Visual C++ 11.0 reports fileno(stdin) == 0 for non-console program

New submission from Mateusz Loskot: In pythonrun.c, there is function initstdio() with the following test: static int initstdio(void) { ... /* Set sys.stdin */ fd = fileno(stdin); /* Under some conditions stdin, stdout and stderr may not be connected * and fileno() may point to

[issue17795] backwards-incompatible change in SysLogHandler with unix domain sockets

Changes by Vinay Sajip : -- assignee: -> vinay.sajip nosy: +vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing list Unsu

[issue17778] Fix test discovery for test_multiprocessing.py

Changes by Richard Oudkerk : -- nosy: +sbt ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue5004] socket.getfqdn() doesn't cope properly with purely DNS-based setups

Stijn Hoop added the comment: So after a good nights sleep: does it not make sense to use the canonical hostname iff the name argument is not present / empty? Otherwise, fall back to the documented steps? That way extra API is avoided, and I can't think of a case where you would rather have my

[issue17468] Generator memory leak

Changes by Antoine Pitrou : -- nosy: +benjamin.peterson, ncoghlan ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue14364] Argparse incorrectly handles '--'

Michele Orrù added the comment: Yes, http://bugs.python.org/file29845/dbldash.patch seems to fix this. Attaching the unittests, and noisying on your issue. -- keywords: +patch Added file: http://bugs.python.org/file29938/issue14364.test.patch ___ Pyth

[issue13922] argparse handling multiple "--" in args improperly

Changes by Michele Orrù : -- nosy: +maker ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.or

[issue17796] No mimetype guessed for tar.xz url

New submission from LCID Fire: mimetypes.guess_type( "http://ftp.gnome.org/pub/GNOME/sources/libxml++/2.36/libxml++-2.36.0.tar.xz";), strict=False ) gives (None, None) -- components: Library (Lib) messages: 187348 nosy: lcid-fire priority: normal severity: normal status: open title: