[issue37732] Possible uninitialized variable in Objects/obmalloc.c

2019-08-16 Thread Hansraj Das
Change by Hansraj Das : -- nosy: +hansrajdas ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyt

[issue37824] IDLE: Handle Shell input warnings properly.

2019-08-16 Thread Terry J. Reedy
Terry J. Reedy added the comment: My first post described master at the time, when SyntaxWarnings were converted to SyntexErrors and shell input warnings, in particular, Deprecation warnings were sent to the console or nowhere. My third post describes the situation with the PR, where Syntax

[issue35805] email package folds msg-id identifiers using RFC2047 encoded words where it must not

2019-08-16 Thread Abhilash Raj
Abhilash Raj added the comment: I am slightly confused if this should be backported to bugfix branches since this is technically a new feature, the ability to parse Message-ID field. I would love to see what David and Barry think about this? -- __

[issue21315] email._header_value_parser does not recognise in-line encoding changes

2019-08-16 Thread Abhilash Raj
Change by Abhilash Raj : -- stage: patch review -> resolved status: open -> closed versions: +Python 3.7, Python 3.8 -Python 3.3, Python 3.4, Python 3.5 ___ Python tracker ___

[issue29412] IndexError thrown on email.message.Message.get

2019-08-16 Thread Abhilash Raj
Change by Abhilash Raj : -- stage: patch review -> resolved status: open -> closed versions: +Python 3.8, Python 3.9 -Python 3.5, Python 3.6 ___ Python tracker ___

[issue37790] subprocess.Popen() is extremely slow

2019-08-16 Thread Alexander Pyhalov
Alexander Pyhalov added the comment: Even if I use import subprocess process = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE, s

[issue37790] subprocess.Popen() is extremely slow

2019-08-16 Thread Alexander Pyhalov
Alexander Pyhalov added the comment: I couldn't reproduce entire test, as wsdiff script is rather large, but here's the simplified version. If Popen is used more often, difference is much more significant. # Using workaround $ python3.5 ~/tmp/1.py 1 10.780487537384033 # Without workaround $

[issue34155] email.utils.parseaddr mistakenly parse an email

2019-08-16 Thread Abhilash Raj
Abhilash Raj added the comment: Created a backport PR for 3.5. -- stage: patch review -> resolved ___ Python tracker ___ ___ Python

[issue34155] email.utils.parseaddr mistakenly parse an email

2019-08-16 Thread Abhilash Raj
Change by Abhilash Raj : -- pull_requests: +15036 stage: resolved -> patch review pull_request: https://github.com/python/cpython/pull/15317 ___ Python tracker ___

[issue34155] email.utils.parseaddr mistakenly parse an email

2019-08-16 Thread Kyle Stanley
Kyle Stanley added the comment: > This is already backported to 3.6. I am not sure about what gets backported > to 3.5 right now, I don't even see a 'Backport to 3.5' label on Github (which > made me think we are discouraged to backport to 3.5). I can work on a manual > backport if needed?

[issue37834] readlink on Windows cannot read app exec links

2019-08-16 Thread Steve Dower
Steve Dower added the comment: > Why group all reparse points under the banner of 'link'? Because for the purposes of the list of changes beneath it, there wasn't any difference (e.g. "traverses any links supported by the OS" is more meaningful to most people, even though both of us would un

[issue37834] readlink on Windows cannot read app exec links

2019-08-16 Thread Eryk Sun
Eryk Sun added the comment: > the '/mnt/c/Document and Settings' junction... though now I try > it that those don't actually work...) The security on compatibility junctions denies everyone read-data (list) access, but in Windows they can still be traversed (e.g. "C:/Documents and Settings/

[issue37834] readlink on Windows cannot read app exec links

2019-08-16 Thread Eryk Sun
Eryk Sun added the comment: > Where "links" are the generic term for the set that includes > "reparse point", "symlink", "mount point", "junction", etc.) Why group all reparse points under the banner of 'link'? If we have a typical HSM reparse point (the vast majority of allocated tags), the

[issue37826] Document PEP 3134 in tutorials/errors.rst

2019-08-16 Thread Abhilash Raj
Change by Abhilash Raj : -- stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___ Python-bugs-list maili

[issue37877] MacOS crash appJar 3.7.3

2019-08-16 Thread Michael Hearn
Michael Hearn added the comment: Hopefully apple will fix this. https://medium.com/@michaeljosephhearn/macos-mojave-crash-6fd1e69d3d34 -- ___ Python tracker ___ __

[issue17305] IDNA2008 encoding is missing

2019-08-16 Thread Christian Heimes
Christian Heimes added the comment: There is no consensus yet. Besides https://pypi.org/project/idna/ we could also consider to wrap libidn2 and ship it. Both PyPI idna and libidn2 have potential licensing issues. I don't like the idea to reinvent the wheel and implement our own idna2008 cod

[issue37877] MacOS crash appJar 3.7.3

2019-08-16 Thread Michael Hearn
Change by Michael Hearn : -- resolution: -> third party stage: -> resolved status: open -> closed ___ Python tracker ___ ___ Pytho

[issue37877] MacOS crash appJar 3.7.3

2019-08-16 Thread Michael Hearn
Michael Hearn added the comment: Thanks for the fast response it uses appJar which is a very similar GUI package but not the same one. AppJar is based of it but I don’t believe it was built off it. On Fri, Aug 16, 2019 at 3:55 PM Ronald Oussoren wrote: > > Ronald Oussoren added the comment:

[issue37732] Possible uninitialized variable in Objects/obmalloc.c

2019-08-16 Thread Hansraj Das
Change by Hansraj Das : -- keywords: +patch pull_requests: +15035 stage: -> patch review pull_request: https://github.com/python/cpython/pull/15309 ___ Python tracker ___

[issue37824] IDLE: Handle Shell input warnings properly.

2019-08-16 Thread Tal Einat
Tal Einat added the comment: I see none of this on latest master (0567786d26348aa7eaf0ab1b5d038fdabe409d92) on Windows 10. Can you specify on what OS and Python branch/version this happens? -- ___ Python tracker

[issue37878] Sub-Interpreters : Document PyThreadState_DeleteCurrent()

2019-08-16 Thread Joannah Nanjekye
Change by Joannah Nanjekye : -- keywords: +patch pull_requests: +15034 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/15315 ___ Python tracker

[issue37878] Sub-Interpreters : Document PyThreadState_DeleteCurrent()

2019-08-16 Thread Joannah Nanjekye
Change by Joannah Nanjekye : -- title: Sub-ineterpreters : Document PyThreadState_DeleteCurrent() ? -> Sub-Interpreters : Document PyThreadState_DeleteCurrent() ___ Python tracker ___

[issue37593] ast.arguments has confusing args/posonlyargs ordering

2019-08-16 Thread Mairi Dulaney
Mairi Dulaney added the comment: The patch that was merged to fix this may have caused a regresion in xonsh. Am working on a reproducer. -- nosy: +mdulaney ___ Python tracker __

[issue37878] Sub-ineterpreters : Document PyThreadState_DeleteCurrent() ?

2019-08-16 Thread Joannah Nanjekye
Change by Joannah Nanjekye : -- title: sub-ineterpreters : Document PyThreadState_DeleteCurrent() ? -> Sub-ineterpreters : Document PyThreadState_DeleteCurrent() ? ___ Python tracker

[issue37877] MacOS crash appJar 3.7.3

2019-08-16 Thread Ronald Oussoren
Ronald Oussoren added the comment: Does this application use Tkinter? If so, this is a duplicate of #37833, and a bug in macOS triggered by (amongst others) anaconda but (AFAIK) not by the Python.org installer. The "logout" is a crash in the WindowServer proces, which is something that sh

[issue37878] sub-ineterpreters : Document PyThreadState_DeleteCurrent() ?

2019-08-16 Thread Joannah Nanjekye
New submission from Joannah Nanjekye : I just noticed that PyThreadState_DeleteCurrent()in Python/pystate.c is not documented. Relevant documentation should go in Doc/c-api/init.rst If no one objects to this. -- assignee: docs@python components: Documentation keywords: easy messages:

[issue37877] MacOS crash appJar 3.7.3

2019-08-16 Thread Michael Hearn
New submission from Michael Hearn : macOS 10.14.6 -anaconda install Python 3.7.3 pip install appJar (0.94.0) On start of app with app.go() a crash will occur causing mac to logout. Python 3.7.2 does not show this behavior Will document on medium. https://raw.githubusercontent.com/borosilic

[issue37835] typing.get_type_hints wrong namespace for forward-declaration of inner class

2019-08-16 Thread Ivan Levkivskyi
Change by Ivan Levkivskyi : -- nosy: +levkivskyi ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail

[issue9949] os.path.realpath on Windows does not follow symbolic links

2019-08-16 Thread Steve Dower
Steve Dower added the comment: Another minor change worth calling out - I added a note to the docs that when a symlink cycle is detected, realpath() could return any member of the cycle, but does not guarantee which one. For our test cases it's generally stable enough, but if you change the

[issue37872] Move _Py_IDENTIFIER statics in Python/import.c to top of the file

2019-08-16 Thread Greg Price
Change by Greg Price : -- components: +Interpreter Core ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https

[issue37872] Move _Py_IDENTIFIER statics in Python/import.c to top of the file

2019-08-16 Thread Greg Price
Change by Greg Price : -- title: Move statics in Python/import.c to top of the file -> Move _Py_IDENTIFIER statics in Python/import.c to top of the file ___ Python tracker ___

[issue37872] Move statics in Python/import.c to top of the file

2019-08-16 Thread Greg Price
Change by Greg Price : -- nosy: +Greg Price ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue37877] MacOS crash appJar 3.7.3

2019-08-16 Thread Michael Hearn
Change by Michael Hearn : -- components: macOS nosy: Michael Hearn, ned.deily, ronaldoussoren priority: normal severity: normal status: open title: MacOS crash appJar 3.7.3 versions: Python 3.7 ___ Python tracker

[issue25299] TypeError: __init__() takes at least 4 arguments (4 given)

2019-08-16 Thread Joannah Nanjekye
Joannah Nanjekye added the comment: > this argument defaults to "None" Am leaning to the fact that this works as required. The documentation does not just say defaults to none. It says: With the 'store_const' and 'append_const' actions, the const keyword argument must be given. For other ac

[issue24754] argparse add_argument with action="store_true", type=bool should not crash

2019-08-16 Thread Joannah Nanjekye
Joannah Nanjekye added the comment: See issue25299 also. -- nosy: +nanjekyejoannah ___ Python tracker ___ ___ Python-bugs-list mail

[issue37834] readlink on Windows cannot read app exec links

2019-08-16 Thread Steve Dower
Steve Dower added the comment: That was a long set of replies, so here's my summary proposed behaviour: (Where "links" are the generic term for the set that includes "reparse point", "symlink", "mount point", "junction", etc.) os.lstat(path): returns attributes for path without traversing an

[issue37834] readlink on Windows cannot read app exec links

2019-08-16 Thread Steve Dower
Steve Dower added the comment: >> I don't want to add any parameters - I want to have predictable and >> reasonable default behaviour. os.readlink() already exists for >> "open reparse point" behaviour. > > I'd appreciate a parameter to always open reparse points, even if a > filter-driver or

[issue37876] Tests for Rot13 codec

2019-08-16 Thread Zeth
New submission from Zeth : Julius Caesar might have written the Rot13 codec, but he forgot to write unit tests, dragging down test coverage. -- components: Tests messages: 349874 nosy: zeth priority: normal pull_requests: 15033 severity: normal status: open title: Tests for Rot13 codec

[issue37874] json traceback on a float

2019-08-16 Thread af
af added the comment: I understand, thanks. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.

[issue37871] 40 * 473 grid of "é" has a single wrong character on Windows

2019-08-16 Thread Steve Dower
Steve Dower added the comment: I'd rather keep encoding incrementally, and reduce the length of each attempt until the last UTF-8 character does not have its top bit set (i.e. is the final character in a multi-byte sequence). Otherwise the people who like to print >2GB worth of data to the c

[issue37874] json traceback on a float

2019-08-16 Thread Eric V. Smith
Eric V. Smith added the comment: I think Serhiy means 1.e-8 is not valid. The spec says that at least one digit is required in the "frac" part, after the decimal. -- nosy: +eric.smith ___ Python tracker __

[issue37875] gzip module any difference for compressing png file in version 2.X and 3.X

2019-08-16 Thread Zachary Ware
Zachary Ware added the comment: Sorry, but your report is unclear. Are you asking if there is any difference, or are you reporting a difference? If the former, this is not the correct forum for such questions, please try python-l...@python.org or StackOverflow instead. If the latter, you'

[issue37824] IDLE: Handle Shell input warnings properly.

2019-08-16 Thread Terry J. Reedy
Terry J. Reedy added the comment: The triple (or double, see below) printing seems like a pure bug to be fixed. A fix could be a separate PR to be merged 'immediately'. But there is a complication with insertion location. My bug claim was based on a one line statement, where 'before' looks

[issue37875] gzip module any difference for compressing png file in version 2.X and 3.X

2019-08-16 Thread Sivaprabu Ganesan
New submission from Sivaprabu Ganesan : gzip module any difference for compressing png file in version 2.X and 3.X -- components: 2to3 (2.x to 3.x conversion tool) messages: 349868 nosy: Sivaprabu Ganesan, sjoerd priority: normal severity: normal status: open title: gzip module any diff

[issue37874] json traceback on a float

2019-08-16 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: 1.0e-8 is not a format for numbers in JSON. See RFC 7159 (https://tools.ietf.org/html/rfc7159.html#section-6): number = [ minus ] int [ frac ] [ exp ] decimal-point = %x2E ; . digit1-9 = %x31-39 ; 1-9 e = %x65 / %x45

[issue37874] json traceback on a float

2019-08-16 Thread af
Change by af : -- type: crash -> behavior ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue37874] json traceback on a float

2019-08-16 Thread af
New submission from af : json.loads traceback with: [In [16]: json.loads("[1.e-8]") --- JSONDecodeError Traceback (most recent call last) in () > 1 json.loads("[1.e-8]") /scr/fonari/2019-4/in

[issue29535] datetime hash is deterministic in some cases

2019-08-16 Thread Tim Peters
Tim Peters added the comment: I'm with Mark: leave numeric hashes alone. There's no reason to change them, and in addition to what Mark wrote it's a positively Good Thing that `hash(i) == i` for all sufficiently small ints. Not only is that efficient to compute, it guarantees there are no

[issue37873] unittest: execute tests in parallel

2019-08-16 Thread D. A. Pellegrino
New submission from D. A. Pellegrino : The unittest documentation makes reference to a potential parallelization feature: "Note that shared fixtures do not play well with [potential] features like test parallelization and they break test isolation. They should be used with care." (https://do

[issue37703] Inconsistent gather with child exception

2019-08-16 Thread Vinay Sharma
Vinay Sharma added the comment: Hi Andrew, Thanks for replying! I understand that the behavior change is not backward compatible. But the current behavior is a bit ambiguous ad mentioned by Dimitri, therefore I have updated the documentation and opened a Pull request. I would be very glad i

[issue37872] Move statics in Python/import.c to top of the file

2019-08-16 Thread Josh Rosenberg
Change by Josh Rosenberg : -- title: Move statitics in Python/import.c to top of the file -> Move statics in Python/import.c to top of the file ___ Python tracker ___ ___

[issue37703] Inconsistent gather with child exception

2019-08-16 Thread Vinay Sharma
Change by Vinay Sharma : -- pull_requests: +15031 stage: -> patch review pull_request: https://github.com/python/cpython/pull/15312 ___ Python tracker ___

[issue37872] Move statitics in Python/import.c to top of the file

2019-08-16 Thread Joannah Nanjekye
Joannah Nanjekye added the comment: Keeping this until the PR https://github.com/python/cpython/pull/15057 is merged. -- ___ Python tracker ___ __

[issue37872] Move statitics in Python/import.c to top of the file

2019-08-16 Thread Joannah Nanjekye
Joannah Nanjekye added the comment: Relevant comment from @gnprice. I think it would be better to make that as a separate change. This style where they're block-local is the way the existing code works, which keeps the diff simple. It's also the way that every use of _Py_IDENTIFIER in thi

[issue37872] Move statitics in Python/import.c to top of the file

2019-08-16 Thread Joannah Nanjekye
New submission from Joannah Nanjekye : Following a PR review here: https://github.com/python/cpython/pull/15057 by @ericsnowcurrently i.e : Please move both these statics to the top of the file (next to CACHEDIR) as globals. To keep the diff in the above PR simple, I am tracking this refacto

[issue37824] IDLE: Handle Shell input warnings properly.

2019-08-16 Thread Tal Einat
Tal Einat added the comment: Il take a look at this. I've very recently gone through the relevant parts of the code and I should be able to figure it it efficiently. -- ___ Python tracker __

[issue37790] subprocess.Popen() is extremely slow

2019-08-16 Thread Antoine Pitrou
Antoine Pitrou added the comment: Can you post a simple script showcasing the performance degradation? -- nosy: +pitrou ___ Python tracker ___

[issue37540] vectorcall: keyword names must be strings

2019-08-16 Thread Jeroen Demeyer
Change by Jeroen Demeyer : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ __

[issue37703] Inconsistent gather with child exception

2019-08-16 Thread Andrew Svetlov
Andrew Svetlov added the comment: The behavior change is not backward compatible, we cannot apply it. Also, please use github pull requests for proposing a patch: https://devguide.python.org/pullrequest/ -- ___ Python tracker

[issue17305] IDNA2008 encoding is missing

2019-08-16 Thread Andrew Svetlov
Andrew Svetlov added the comment: There is no consensus yet, IMHO. There is a lack of resources for the issue. -- ___ Python tracker ___ __

[issue37540] vectorcall: keyword names must be strings

2019-08-16 Thread miss-islington
miss-islington added the comment: New changeset 0567786d26348aa7eaf0ab1b5d038fdabe409d92 by Miss Islington (bot) (Jeroen Demeyer) in branch 'master': bpo-37540: vectorcall: keyword names must be strings (GH-14682) https://github.com/python/cpython/commit/0567786d26348aa7eaf0ab1b5d038fdabe409d

[issue29535] datetime hash is deterministic in some cases

2019-08-16 Thread Mark Dickinson
Mark Dickinson added the comment: > Why so? Python's hash needs to obey the invariant that `hash(x) == hash(y)` for any two hashable objects with `x == y`. That makes life particularly hard for numeric types, because there are a number of *different* numeric types where equality can hold ac

[issue37703] Inconsistent gather with child exception

2019-08-16 Thread Vinay Sharma
Change by Vinay Sharma : Added file: https://bugs.python.org/file48548/gather_cancel_code.patch ___ Python tracker ___ ___ Python-bugs-list

[issue37703] Inconsistent gather with child exception

2019-08-16 Thread Vinay Sharma
Vinay Sharma added the comment: Hi Dimitri, You are right, gather.cancel() doesn't work once it has propagated an exception. This happens because after propagating the exception to the caller, gather is marked as done. And cancel doesn't work after a Future object has been marked done. You c

[issue37256] urllib.request.Request documentation erroneously refers to the "final two"

2019-08-16 Thread miss-islington
miss-islington added the comment: New changeset f9919121460bffc04f935dbdb85f0af3ffbd3ddf by Miss Islington (bot) in branch '3.8': bpo-37256: Wording in Request class docs (GH-14792) https://github.com/python/cpython/commit/f9919121460bffc04f935dbdb85f0af3ffbd3ddf -- ___

[issue37256] urllib.request.Request documentation erroneously refers to the "final two"

2019-08-16 Thread Senthil Kumaran
Senthil Kumaran added the comment: Thank you for the patch, CuriousLearner. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker