[issue42778] Add follow_symlinks=True parameter to both os.path.samefile() and Path.samefile()

[Tom Hale]

Tom Hale  added the comment:

In summary:

The underlying os.stat() takes a follow_symlinks=True parameter but it can't be 
set to False when trying to samefile() two symbolic links.

--
title: Add follow_symlinks=True to {os.path,Path}.samefile -> Add 
follow_symlinks=True parameter to both os.path.samefile() and Path.samefile()

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42778] Add follow_symlinks=True to {os.path,Path}.samefile

[Tom Hale]

New submission from Tom Hale :

The os.path and Path implementations of samefile() do not allow comparisons of 
symbolic links:

% mkdir empty && chdir empty
% ln -s non-existant broken
% ln broken lnbroken
% ls -i # Show inode numbers
19325632 broken@  19325632 lnbroken@
% Yup, they are the same file... but...
% python -c 'import os; print(os.path.samefile("lnbroken", "broken", 
follow_symlinks=False))'
Traceback (most recent call last):
  File "", line 1, in 
TypeError: samefile() got an unexpected keyword argument 'follow_symlinks'
% python -c 'import os; print(os.path.samefile("lnbroken", "broken"))'
Traceback (most recent call last):
  File "", line 1, in 
  File "/usr/lib/python3.8/genericpath.py", line 100, in samefile
s1 = os.stat(f1)
FileNotFoundError: [Errno 2] No such file or directory: 'lnbroken'
%

Both samefile()s use os.stat under the hood, but neither allow setting  
os.stat()'s `follow_symlinks` parameter.

https://docs.python.org/3/library/os.html#os.stat

https://docs.python.org/3/library/os.path.html#os.path.samefile

https://docs.python.org/3/library/pathlib.html#pathlib.Path.samefile

--
components: Library (Lib)
messages: 383965
nosy: Tom Hale
priority: normal
severity: normal
status: open
title: Add follow_symlinks=True to {os.path,Path}.samefile
type: behavior
versions: Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42560] Improve Tkinter Documentation

[FredInChina]

FredInChina  added the comment:

I agree with @MasonGinter, the documentation is somewhat lacking. Relying on 
third parties is fragile - we were fortunate that the University of New Mexico 
Tech docs maintained by Pr. John W. Shipman were 
[salvaged](https://anzeljg.github.io/rin2/book2/2405/docs/tkinter/index.html) 
after his premature death; yet, some important example files were lost.

We now see [effbot.org](effbot.org) "taking a leave"... What other warnings do 
we need?


> "we don't know what Tk version to write them for".  
I don't understand why @epaine; shouldn't the docs be made available for the 
currently maintained version of python?

--
nosy: +ReblochonMasque

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7057] tkinter doc: more 3.x updates

[FredInChina]

FredInChina  added the comment:

Are there good reasons not to close this issue?
It is marked as "easy", but it seems that the problems in reference have either 
been fixed, or died of old age.

--
nosy: +ReblochonMasque

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42775] __init_subclass__ should be called in __init__

[Ethan Furman]

Ethan Furman  added the comment:

Guido, I just wanted to get it all in place while it was fresh in my mind.  
Actual code tends to make a discussion easier.  I'll make sure and transcribe 
any relevant discussion from python-dev to here.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42776] The string find method shows the problem

[ye andy]

ye andy  added the comment:

When I say True here, I'm not talking about querying, I'm talking about
syntax, as far as I know, document is supposed to support keyword
arguments, but it doesn't

Steven D'Aprano  于2020年12月29日周二 上午11:36写道：

>
> Steven D'Aprano  added the comment:
>
> # True
> data.find('a', 0)
>
> That will return 0, not True.
>
>
> # False
> data.find('a', start=0)
>
>
> And that will raise TypeError, not return False.
>
>
> What exactly are you reporting here? I have read your bug report, and
> looked at the screen shot, and I have no idea what problem you think you
> have found or what you want to change about the documentation.
>
> --
> nosy: +steven.daprano
>
> ___
> Python tracker 
> 
> ___
>

--
nosy: +andy.ye.jx

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42775] __init_subclass__ should be called in __init__

[Guido van Rossum]

Guido van Rossum  added the comment:

Whoa, you start a discussion on python-dev and another on bpo? That sounds a 
bit hasty.

--
nosy: +gvanrossum

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42775] __init_subclass__ should be called in __init__

[Ethan Furman]

Change by Ethan Furman :


--
keywords: +patch
pull_requests: +22829
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/23986

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42775] __init_subclass__ should be called in __init__

[Ethan Furman]

Change by Ethan Furman :


--
nosy: +ncoghlan

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[Zachary Ware]

Zachary Ware  added the comment:

Thanks for the report, bazwal; for the patch, Zackery; and for the ping Ammar :)

--
nosy: +zach.ware
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed
type: enhancement -> behavior

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue8595] Explain the default timeout in http-client-related libraries

[Senthil Kumaran]

Change by Senthil Kumaran :


--
assignee: docs@python -> orsenthil
versions: +Python 3.10 -Python 2.6, Python 2.7, Python 3.1

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue8481] doc: ctypes no need to explicitly allocate writable memory with Structure

[Senthil Kumaran]

Senthil Kumaran  added the comment:

This is a not a bug, it asks for parameter types to explained. API 
documentation reference should provide the guidance.

--
nosy: +orsenthil
resolution:  -> wont fix
stage:  -> resolved
status: open -> closed
versions: +Python 3.10 -Python 2.6, Python 2.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue8822] datetime naive and aware types should have a well-defined definition that can be cross-referenced

[Senthil Kumaran]

Change by Senthil Kumaran :


--
versions: +Python 3.10 -Python 2.6, Python 2.7, Python 3.1, Python 3.2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1398781] Example in section 5.3 "Pure Embedding" doesn't work.

[Senthil Kumaran]

Change by Senthil Kumaran :


--
keywords:  -easy
versions: +Python 3.10 -Python 2.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[miss-islington]

miss-islington  added the comment:


New changeset 0b43778b3cd222761beb850178492b6bd0ea2370 by Miss Islington (bot) 
in branch '3.9':
bpo-42770: Fix a typo in the email.headerregistry docs (GH-23982)
https://github.com/python/cpython/commit/0b43778b3cd222761beb850178492b6bd0ea2370


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[miss-islington]

miss-islington  added the comment:


New changeset e11639880a73f30b4009efa8d14c350932e35332 by Miss Islington (bot) 
in branch '3.8':
bpo-42770: Fix a typo in the email.headerregistry docs (GH-23982)
https://github.com/python/cpython/commit/e11639880a73f30b4009efa8d14c350932e35332


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42777] WindowsPath does not implement is_mount but ntpath implements and offers a ismount method

[David]

New submission from David :

pathlib.WindowsPath[0] does not implement is_mount but ntpath implements and 
offers a ismount[1] method. Perhaps WindowsPath is_mount can make use of 
ntpath.ismount ?



[0] https://github.com/python/cpython/blob/master/Lib/pathlib.py#L1578
[1] https://github.com/python/cpython/blob/master/Lib/ntpath.py#L248

--
messages: 383955
nosy: db
priority: normal
severity: normal
status: open
title: WindowsPath does not implement is_mount but ntpath implements and offers 
a ismount method

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[miss-islington]

Change by miss-islington :


--
pull_requests: +22827
pull_request: https://github.com/python/cpython/pull/23984

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[miss-islington]

Change by miss-islington :


--
pull_requests: +22828
pull_request: https://github.com/python/cpython/pull/23985

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[miss-islington]

miss-islington  added the comment:


New changeset c56988b88fecf6dc70f039704fda6051a0754db1 by Zackery Spytz in 
branch 'master':
bpo-42770: Fix a typo in the email.headerregistry docs (GH-23982)
https://github.com/python/cpython/commit/c56988b88fecf6dc70f039704fda6051a0754db1


--
nosy: +miss-islington

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42717] The python interpreter crashed with "_enter_buffered_busy"

[Xinmeng Xia]

Xinmeng Xia  added the comment:

Could we try to limit the number of thread or state or something? I mean if we 
set parameter of "range", for example, to 1000 or less here, the crash will no 
longer happen. I think the parser can not handle too heavy loop so that it 
crashes.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42763] Exposing a race in the "_warnings" resulting Python parser crash

[Xinmeng Xia]

Xinmeng Xia  added the comment:

Thank you, but I don't think this is a duplicate of issue42717. This crash is 
probably caused by the parameter "target" of Thread. "Target" accept the 
"callable object". Defaults to None. In this program, it's assigned to a "Int 
object". I think a pre-checking of parameter for such error should be added. 



Minimal test case should be:
===
import threading
for i in range(10):
t = threading.Thread(target=1, daemon=True).start()  
= 



In fact, I try this one. No loop, It sometime crashes the parser.(not always, 
twice in ten times)
==
import threading
t = threading.Thread(target=1, daemon=True).start()
==

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42776] The string find method shows the problem

[Steven D'Aprano]

Steven D'Aprano  added the comment:

# True
data.find('a', 0)

That will return 0, not True.


# False
data.find('a', start=0)


And that will raise TypeError, not return False.


What exactly are you reporting here? I have read your bug report, and looked at 
the screen shot, and I have no idea what problem you think you have found or 
what you want to change about the documentation.

--
nosy: +steven.daprano

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42762] infinite loop resulted by "yield"

[Xinmeng Xia]

Xinmeng Xia  added the comment:

Thanks for your kind explanation! My description is a little confusing. Sorry 
about that. The problem here is that the program should stop when the exception 
is caught whatever the exception is. (I think bpo-25612 (#1773) fixed exception 
selection problems and right exception can be caught) The fact is this program 
will fall into an infinite loop. Error messages are printed in a dead loop. The 
program doesn't stop. This is not normal. There will be no loops in Python 3.5 
and 3.6 (Attached output in Python 3.5 and 3.6). 

 

Output on Python 3.5,3.6 (all errors are printed without any loop )
-
Exception ignored in: 
RuntimeError: generator ignored GeneratorExit
Traceback (most recent call last):
  File "/home/xxm/Desktop/nameChanging/report/test1.py", line 248, in 
print(i)
NameError: name 'i' is not defined
Exception ignored in: 
RuntimeError: generator ignored GeneratorExit
Exception ignored in: 
RuntimeError: generator ignored GeneratorExit


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42776] The string find method shows the problem

[andy ye]

New submission from andy ye :

data = 'abcddd'
# True
data.find('a', 0)

# False
data.find('a', start=0)


# document
class str(obj):

def find(self, sub, start=None, end=None): # real signature unknown; 
restored from __doc__
"""
S.find(sub[, start[, end]]) -> int

Return the lowest index in S where substring sub is found,
such that sub is contained within S[start:end].  Optional
arguments start and end are interpreted as in slice notation.

Return -1 on failure.
"""
return 0

--
assignee: docs@python
components: Documentation
files: 截屏2020-12-29 上午10.57.53.png
messages: 383949
nosy: andyye, docs@python
priority: normal
severity: normal
status: open
title: The string find method shows the problem
versions: Python 3.6
Added file: https://bugs.python.org/file49704/截屏2020-12-29 上午10.57.53.png

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42775] __init_subclass__ should be called in __init__

[Ethan Furman]

Ethan Furman  added the comment:

My understanding of using keywords in class headers

  class MyClass(SomeBaseClass, setting='maybe'):
...

is that the keywords would get passed into the super classes 
`__init_subclass__`  (`SomeBaseClass` and `setting`).

However, in the cases of 

- test_descr.py
- test_py_compile.py
- typing.py

that wasn't happening -- until the initial patch of moving the calls from 
`type.__new__` to `type.__init__`.

An `__init__` has been added in those three locations to discard the keyword 
arguments being passed in.

--
nosy: +rhettinger, stutzbach

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42775] __init_subclass__ should be called in __init__

[Ethan Furman]

New submission from Ethan Furman :

PEP 487 introduced __init_subclass__ and __set_name__, and both of those were 
wins for the common cases of metaclass usage.

Unfortunately, the implementation of PEP 487 with regards to __init_subclass__ 
has made the writing of correct metaclasses significantly harder, if not 
impossible.

The cause is that when a metaclass calls type.__new__ to actually create the 
class, type.__new__ calls the __init_subclass__ methods of the new class' 
parents, passing it the newly created, but incomplete, class.  In code:

```
class Meta(type):
#
def __new__(mcls, name, bases, namespace, **kwds):
# create new class, which will call __init_subclass__ and __set_name__
new_class = type.__new__(mcls, name, bases, namespace, **kwds)
# finish setting up class
new_class.some_attr = 9
```

As you can deduce, when the parent __init_subclass__ is called with the new 
class, `some_attr` has not been added yet -- the new class is incomplete.

For Enum, this means that __init_subclass__ doesn't have access to the new 
Enum's members (they haven't beet added yet).

For ABC, this means that __init_subclass__ doesn't have access to 
__abstract_methods__ (it hasn't been created yet).

Because Enum is pure Python code I was able to work around it:
- remove new __init_subclass__ (if it exists)
- insert dummy class with a no-op __init_subclass__
- call type.__new__
- save any actual __init_subclass__
- add back any new __init_subclass__
- rewrite the new class' __bases__, removing the no-op dummy class
- finish creating the class
- call the parent __init_subclass__ with the now complete Enum class

I have not been able to work around the problem for ABC.

The solution would seem to be to move the calls to __init_subclass__ and 
__set_names__ to type.__init__.

--
assignee: ethan.furman
components: Interpreter Core
messages: 383946
nosy: ethan.furman, serhiy.storchaka
priority: high
severity: normal
stage: needs patch
status: open
title: __init_subclass__ should be called in __init__
type: behavior
versions: Python 3.10

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35815] Able to instantiate a subclass with abstract methods from __init_subclass__ of the ABC

[Ethan Furman]

Ethan Furman  added the comment:

If the patch in issue42775 is committed, this problem will be solved.

--
assignee:  -> ethan.furman
superseder:  -> __init_subclass__ should be called in __init__

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42740] typing.py get_args and get_origin should support PEP 604 and 612

[Guido van Rossum]

Change by Guido van Rossum :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42740] typing.py get_args and get_origin should support PEP 604 and 612

[Guido van Rossum]

Guido van Rossum  added the comment:


New changeset efb1f0918fad2ba3346a986b0e958dc5753a7bbe by Ken Jin in branch 
'master':
bpo-42740: Support PEP 604, 612 for typing.py get_args and get_origin (GH-23942)
https://github.com/python/cpython/commit/efb1f0918fad2ba3346a986b0e958dc5753a7bbe


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42774] 'ipaddress' module, bad result for 'is_private' on "192.0.0.0"

[Trevor Marvin]

Change by Trevor Marvin :


--
resolution:  -> not a bug
stage:  -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42753] "./configure" linux chrome beta

[Gerrik Labra]

Gerrik Labra  added the comment:

Understood. Thank you.

On Mon, Dec 28, 2020 at 10:50 AM Eric V. Smith 
wrote:

>
> Eric V. Smith  added the comment:
>
> This appears to not be a bug with Python, so I'm going to close this. If
> you're having additional build problems, I suggest you ask for help on the
> python-list mailing list, or maybe on Stack Overflow.
>
> --
> resolution:  -> not a bug
> stage:  -> resolved
> status: open -> closed
> type:  -> behavior
>
> ___
> Python tracker 
> 
> ___
>

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42765] Introduce new data model method __iter_items__

[Eric V. Smith]

Change by Eric V. Smith :


--
nosy: +eric.smith

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42774] 'ipaddress' module, bad result for 'is_private' on "192.0.0.0"

[Eric V. Smith]

Eric V. Smith  added the comment:

The ipaddress documentation references 
https://www.iana.org/assignments/iana-ipv4-special-registry/iana-ipv4-special-registry.xhtml,
 which says that 192.0.0.0/29 is reserved and not globally reachable. I think 
in that sense, it's a private address.

--
nosy: +eric.smith

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42774] 'ipaddress' module, bad result for 'is_private' on "192.0.0.0"

[Trevor Marvin]

New submission from Trevor Marvin :

Tested on Python 3.6.9 with "ipaddress" module, module version 1.0.

ipaddress.ip_address('192.0.0.0').is_private

Incorrectly returns as 'True'.  Per RFC 1918 / BCP 5, section 3, the private 
IPv4 space sarting with '192' is only '192.168.0.0/16'.

--
components: Library (Lib)
messages: 383942
nosy: trevormarvin
priority: normal
severity: normal
status: open
title: 'ipaddress' module, bad result for 'is_private' on "192.0.0.0"
type: behavior
versions: Python 3.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42773] build.yml workflow not testing on pushes

[Ammar Askar]

Ammar Askar  added the comment:

Any commit that gets to pushed to the master, 3.9, 3.8 and 3.7 branches.

e.g all the commits here https://github.com/python/cpython/commits/master, they 
end up failing the "Check for source changes" step and skipping even when 
there's non-documentation changes.

--
versions: +Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40522] [subinterpreters] Get the current Python interpreter state from Thread Local Storage (autoTSSkey)

[STINNER Victor]

STINNER Victor  added the comment:

Mark Shannon experiment using __thread:

* 
https://mail.python.org/archives/list/python-...@python.org/thread/RPSTDB6AEMIACJFZKCKIRFTVLAJQLAS2/

* 
https://github.com/python/cpython/compare/master...markshannon:threadstate_in_tls

He added " extern __thread struct _ts *_Py_tls_tstate;" to 
Include/cpython/pystate.h.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[Zackery Spytz]

Zackery Spytz  added the comment:

I've created a fix for this error.

--
versions: +Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Raymond Hettinger]

Raymond Hettinger  added the comment:

> Should I restore that optimization in issue37319?

Yes, but to fix the bug it needs to occur earlier in the code (currently line 
314):

if stop is None and step is _ONE: <-- Line 314
   ^  <-- Add this   

And again later:
 if istep == 1 and width > 0:   <-- Line 348
 ^-- change to _ONE

As a standalone optimization, it wasn't worth it, but as a way to fix the bug 
without a regression, it is reasonable.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[Zackery Spytz]

Change by Zackery Spytz :


--
keywords: +patch
nosy: +ZackerySpytz
nosy_count: 3.0 -> 4.0
pull_requests: +22826
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/23982

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42773] build.yml workflow not testing on pushes

[STINNER Victor]

STINNER Victor  added the comment:

> build.yml workflow not testing on pushes

What does it mean: "on pushes"?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42754] Unpacking of literals inside other literals should be optimised away by the compiler

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

> This is a similar case. I am -1 for this optimization.

Yeah, I concur. One of the major drawbacks of this is that the normal case 
where this happens involves a Load():

z = "something"
x = ["a", *z, "b"]

I see almost no reason to nest the literals in this case, especially in a 
context sensitive scope. I am closing the draft PR and the issue.

Thanks Patrick for the proposal, though!

--
resolution:  -> rejected
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42393] Raise overflow errors iso. deprecation warnings in socket.htons and socket.ntohs (was deprecated in 3.7)

[Erlend Egeberg Aasland]

Change by Erlend Egeberg Aasland :


--
pull_requests: +22825
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/23980

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35943] PyImport_GetModule() can return partially-initialized module

[Cebtenzzre]

Change by Cebtenzzre :


--
nosy: +cebtenzzre

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue41567] multiprocessing.Pool from concurrent threads failure on 3.9.0rc1

[Cebtenzzre]

Change by Cebtenzzre :


--
nosy: +cebtenzzre

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42243] Don't access the module dictionary directly

[Erlend Egeberg Aasland]

Change by Erlend Egeberg Aasland :


--
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42243] Don't access the module dictionary directly

[Erlend Egeberg Aasland]

Change by Erlend Egeberg Aasland :


--
resolution:  -> wont fix

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42773] build.yml workflow not testing on pushes

[Ammar Askar]

New submission from Ammar Askar :

It looks like on pushes to Github, we currently aren't running tests. (Though 
this isn't too big of a concern since they get run on pull requests).

Here's an example of a recent run 
https://github.com/python/cpython/runs/1609911031

```
fatal: ambiguous argument 'origin/..': unknown revision or path not in the 
working tree.
Use '--' to separate paths from revisions, like this:
'git  [...] -- [...]'
```

--
components: Build
messages: 383935
nosy: FFY00, Mariatta, ammar2, vstinner
priority: normal
severity: normal
status: open
title: build.yml workflow not testing on pushes
type: behavior
versions: Python 3.10, Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42754] Unpacking of literals inside other literals should be optimised away by the compiler

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

I do not think there are serious raesons for adding this optimization. The 
Python compiler was intentionally made simple for maintainability. Constant 
folding supports only base arithmetic and bits operations because they are 
often used in constant expressions (like 2**32-1 or 1<<18) and indexing because 
of b'A'[0]. Neither comparisons, nor boolean operators with constants are 
optimized, because such expression are uncommon, and the maintaining cost 
overdraws benefit.

This is a similar case. I am -1 for this optimization.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42765] Introduce new data model method __iter_items__

[Steven D'Aprano]

Steven D'Aprano  added the comment:

Hi Richard,


> Also, using _asdict() seems strange as an exposed API, since it's an 
> underscore method and users hence might not be inclined to use it.

I don't consider this a strong argument. Named tuple in general has to use a 
naming convention for public methods that cannot clash with field names, hence 
the single underscore, but your concrete named tuple class can offer any 
methods you like since you know which field names are used and which are not. 
Just add a public method "asdict" or any name you prefer, and delegate to the 
single underscore method.



> It would be nicer to have my type just return this appropriate dict when 
> invoking dict(BasicStats(…)).

As we speak there is a discussion on Python-Ideas about this.

https://mail.python.org/archives/list/python-id...@python.org/thread/2HMRGJ672NDZJZ5PVLMNVW6KP7OHMQDI/#UYDIPMY2HXGL4OLEEFXBTZ2T4CK6TSVU

Your input would be appreciated.


> This would require me to override the __iter__() method to yield key / value 
> tuples for the dict.

The dict constructor does not require that. See discussion on the thread above.

If you search the Python-Ideas archives, I am sure you will find past proposals 
for a `__json__` protocol. If I recall correctly, there was some concern about 
opening the flood-gates for dunder protocols (will this be followed with 
demands for __yaml__, __xml__, __cson__, __toml__, etc?) but perhaps the time 
is right to revisit this idea.

--
nosy: +steven.daprano

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42754] Unpacking of literals inside other literals should be optimised away by the compiler

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

I have added a draft PR on how the idea would look like so we can discuss with 
a specific proposal.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42754] Unpacking of literals inside other literals should be optimised away by the compiler

[Pablo Galindo Salgado]

Change by Pablo Galindo Salgado :


--
keywords: +patch
pull_requests: +22824
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/23979

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

> Another solution is to use an identity test for the step argument

Should I restore that optimization in issue37319?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Raymond Hettinger]

Raymond Hettinger  added the comment:

Another solution is to use an identity test for the step argument

_one = 1

class Random:
def randrange(start, stop=None, step=_one):
...
if stop is None and step is _one:
if istart > 0:
return self._randbelow(istart)

This has the advantage of keeping the API unchanged while still keeping the 
fast path fast.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42739] Crash when try to disassemble bogus code object

[Ammar Askar]

Ammar Askar  added the comment:

This seems to be part 2 of the problems Mark mentioned in issue42562. Namely in 
this case the `co_lnotab` accessor uses PyLineTable_NextAddressRange which has 
that assertion.

--
nosy: +ammar2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Raymond Hettinger]

Raymond Hettinger  added the comment:

Note, we can't actually use "self.choice(range(*args))" because the underlying 
len() call can Overflow.  If it does, the components need to be computed 
manually.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[Ammar Askar]

Ammar Askar  added the comment:

Confirmed, the attribute under 
https://docs.python.org/3.10/library/email.headerregistry.html#email.headerregistry.ContentDispositionHeader
 is incorrect.

bazwal, would you like to propose a patch to fix this? The code is here: 
https://github.com/python/cpython/blob/master/Doc/library/email.headerregistry.rst

--
nosy: +ammar2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42771] Implement interactive hotkey, Ctrl+L to clear the console in Windows.

[Eryk Sun]

Eryk Sun  added the comment:

> As the legacy cmd.exe can be easily replaced, that leaves python.exe. 

python.exe is a console application, which attaches to a console session. Since 
Windows 7, each console session is hosted by an instance of conhost.exe.

The CMD shell (cmd.exe) is a console application, the same as python.exe. It 
doesn't matter whether Python allocates a new console or inherits it from 
another process such as cmd.exe.

> Likely needs to be configured via readline or its analog used under 
> Windows.

pyreadline implements readline for the Windows console API via ctypes. It 
supports Ctrl+L to clear the screen, including the scrollback. PSReadLine in 
PowerShell implements Ctrl+L without clearing the scrollback. That's a better 
example to follow if someone wants to take up the ambitious project of 
supporting readline for Windows in the standard library.

--
nosy: +eryksun

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42740] typing.py get_args and get_origin should support PEP 604 and 612

[miss-islington]

miss-islington  added the comment:


New changeset 03e571f1d5f3e7a11f33bb9f47dd4acedf17b166 by Miss Islington (bot) 
in branch '3.9':
bpo-42740: Fix get_args for PEP 585 collections.abc.Callable (GH-23963)
https://github.com/python/cpython/commit/03e571f1d5f3e7a11f33bb9f47dd4acedf17b166


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42048] Document Argument Clinic's defining_class converter

[Erlend Egeberg Aasland]

Change by Erlend Egeberg Aasland :


--
keywords: +patch
pull_requests: +22823
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/23978

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42740] typing.py get_args and get_origin should support PEP 604 and 612

[miss-islington]

Change by miss-islington :


--
pull_requests: +22822
pull_request: https://github.com/python/cpython/pull/23977

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42740] typing.py get_args and get_origin should support PEP 604 and 612

[miss-islington]

miss-islington  added the comment:


New changeset 4140f10a16f06c32fd49f9e21fb2a53abe7357f0 by Ken Jin in branch 
'master':
bpo-42740: Fix get_args for PEP 585 collections.abc.Callable (GH-23963)
https://github.com/python/cpython/commit/4140f10a16f06c32fd49f9e21fb2a53abe7357f0


--
nosy: +miss-islington

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42767] Review usage of atomic variables in signamodule.c

[Ammar Askar]

Ammar Askar  added the comment:

> For me, the most surprising part is Handlers[signum].tripped which is 
> declared as volatile *and* an atomic variable.

I think it's just following this part of the C spec (Some background in 
bpo-12060):

> or refers to any object with static storage duration other than by assigning 
> a value to a static storage duration variable of type volatile sig_atomic_t. 
> Furthermore, if such a call fails, the value of errno is unspecified.

https://pubs.opengroup.org/onlinepubs/007904875/functions/sigaction.html / 
https://wiki.sei.cmu.edu/confluence/display/c/SIG31-C.+Do+not+access+shared+objects+in+signal+handlers

> I'm not sure if Handlers[signum].func must be volatile neither.

I think your assessment here is right considering it's never used in the signal 
handler itself.

> Also, wakeup.fd is declared as sig_atomic_t on Unix. Could we use an atomic 
> variable instead, or is it important to use "sig_atomic_t"?

Again looking at the 
https://wiki.sei.cmu.edu/confluence/display/c/SIG31-C.+Do+not+access+shared+objects+in+signal+handlers
 recommendation it seems like using an atomic variable is fine but only if it's 
lock-free.

--
nosy: +ammar2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Raymond Hettinger]

Change by Raymond Hettinger :


--
nosy: +tim.peters

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Raymond Hettinger]

Raymond Hettinger  added the comment:

One fix is to move up the code for converting step to istep and then modify the 
early out test to:  

if stop is None and istep == 1:
if istart > 0:
return self._randbelow(istart)

That would have the downside of slowing down the common case.

Another possibility is changing the default step to a sentinel object and just 
testing for that:

if stop is None and step is sentinel:
if istart > 0:
return self._randbelow(istart)

That would be user visible in the function signature but it would run fast.

We could bite the bullet and fully harmonize the randrange() signature with 
range().  That would entail switching to *args and no longer accepting keyword 
arguments:

 def randrange(self, /, *args):
 "Choose random item from range(stop) or range(start, stop[, step])."
 return self.choice(range(*args))

This would most closely match user expectations but is potentially a breaking 
change for existing code that uses keyword arguments.  Such code probably 
exists but is probably not common.

For speed, the actual implementation could still have fast paths for common 
cases.

For help() and tooltips, we could add a __text_signature__ to cover-up the 
*args.  However, signature objects currently aren't capable of describing 
range().  The best we could do is:

randrange.__text_signature__ = '($self, start, stop, step, /)'

That would give help() that looks like this:

>>> help(randrange)
Help on method randrange in module Random:

randrange(start, stop, step, /) method of __main__.R instance
Choose random item from range(stop) or range(start, stop[, step]).

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42762] infinite loop resulted by "yield"

[Steve Stagg]

Steve Stagg  added the comment:

Behaviour was changed in this commit:
---
commit ae3087c6382011c47db82fea4d05f8bbf514265d
Author: Mark Shannon 
Date:   Sun Oct 22 22:41:51 2017 +0100

Move exc state to generator. Fixes bpo-25612 (#1773)

Move exception state information from frame objects to coroutine 
(generator/thread) object where it belongs.

---

I'm honestly not sure that the older behaviour was 'better' than current.  But 
I don't know the defined behaviours well enough to be certain

--
nosy: +stestagg

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42771] Implement interactive hotkey, Ctrl+L to clear the console in Windows.

[Mike Miller]

Mike Miller  added the comment:

I found an implementation of this for Windows in case it is needed.  Not sure 
if it is the best way to do it, as the Console API is rather clumsy.  However 
this one works to my knowledge:

https://github.com/tartley/colorama/blob/master/colorama/winterm.py#L111

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Raymond Hettinger]

Change by Raymond Hettinger :


--
assignee: rhettinger -> 
versions: +Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Raymond Hettinger]

Change by Raymond Hettinger :


--
assignee:  -> rhettinger

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42772] randrange() mishandles step when stop is None

[Raymond Hettinger]

New submission from Raymond Hettinger :

When stop is None, the step argument is ignored:

>>> randrange(1000, None, 100)
651
>>> randrange(1000, step=100)
673

--
components: Library (Lib)
messages: 383919
nosy: rhettinger, serhiy.storchaka
priority: normal
severity: normal
status: open
title: randrange() mishandles step when stop is None
type: behavior
versions: Python 3.10

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37319] Deprecate using random.randrange() with non-integers

[Raymond Hettinger]

Raymond Hettinger  added the comment:

I merged in PR 23064. If you want to make further modifications or improvements 
that would be welcome.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42771] Implement interactive hotkey, Ctrl+L to clear the console in Windows.

[Mike Miller]

New submission from Mike Miller :

The Ctrl+L as clear-screen hotkey is supported just about everywhere, Unix and 
Windows, with the exceptions of cmd.exe and python.exe interactive mode.

As the legacy cmd.exe can be easily replaced, that leaves python.exe.  Likely 
needs to be configured via readline or its analog used under Windows.

Documenting it would be good too.  Am happy to help, write, or test.

--
components: Windows
messages: 383917
nosy: mixmastamyk, paul.moore, steve.dower, tim.golden, zach.ware
priority: normal
severity: normal
status: open
title: Implement interactive hotkey, Ctrl+L to clear the console in Windows.
type: enhancement

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42222] Modernize integer test/conversion in randrange()

[Raymond Hettinger]

Change by Raymond Hettinger :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42222] Modernize integer test/conversion in randrange()

[Raymond Hettinger]

Raymond Hettinger  added the comment:


New changeset a9621bb301dba44494e81edc00e3a3b62c96af26 by Raymond Hettinger in 
branch 'master':
bpo-4: Modernize integer test/conversion in randrange() (#23064)
https://github.com/python/cpython/commit/a9621bb301dba44494e81edc00e3a3b62c96af26


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42763] Exposing a race in the "_warnings" resulting Python parser crash

[Steve Stagg]

Steve Stagg  added the comment:

Looks like a duplicate of issue42717.  

Causing a daemonic thread to terminate while doing IO will trigger the above 
abort

--
nosy: +stestagg

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42753] "./configure" linux chrome beta

[Eric V. Smith]

Eric V. Smith  added the comment:

This appears to not be a bug with Python, so I'm going to close this. If you're 
having additional build problems, I suggest you ask for help on the python-list 
mailing list, or maybe on Stack Overflow.

--
resolution:  -> not a bug
stage:  -> resolved
status: open -> closed
type:  -> behavior

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42717] The python interpreter crashed with "_enter_buffered_busy"

[Steve Stagg]

Steve Stagg  added the comment:

I think the problem here is that the issue can only really be detected late on 
during interpreter shutdown.

This makes recovery very hard to do.  Plus the thread termination has left 
shared state in an unmanaged condition, so it's super dangerous to re-enter 
python again

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42754] Unpacking of literals inside other literals should be optimised away by the compiler

[Batuhan Taskaya]

Batuhan Taskaya  added the comment:

We could possibly fold this at the AST optimizer, though I am not sure whether 
this worths anything as an optimization since it is a real obscure pattern. 
I've only found 2 occurrences (both from the test suite of black) on a 
relatively ~big dataset of python source code.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42754] Unpacking of literals inside other literals should be optimised away by the compiler

[Batuhan Taskaya]

Change by Batuhan Taskaya :


--
Removed message: https://bugs.python.org/msg383911

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42754] Unpacking of literals inside other literals should be optimised away by the compiler

[Batuhan Taskaya]

Batuhan Taskaya  added the comment:

We could possibly fold this at the AST optimizer, though I am not sure whether 
this worths anything as an optimization since it is a real obscure pattern. 
I've only found 2 occurrences (both from a test set on a relatively ~big 
dataset of source code.

--
nosy: +Mark.Shannon, pablogsal, serhiy.storchaka

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42770] Typo in email.headerregistry docs

[bazwal]

New submission from bazwal :

The section for class email.headerregistry.ContentDispositionHeader has a typo 
in an attribute name: "content-disposition" should be corrected to 
"content_disposition".

--
assignee: docs@python
components: Documentation
messages: 383910
nosy: bazwal, docs@python
priority: normal
severity: normal
status: open
title: Typo in email.headerregistry docs
type: enhancement
versions: Python 3.10, Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42765] Introduce new data model method __iter_items__

[Raymond Hettinger]

Change by Raymond Hettinger :


--
nosy: +bob.ippolito

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42765] Introduce new data model method __iter_items__

[Raymond Hettinger]

Raymond Hettinger  added the comment:

This core of this idea is plausible.  It is a common problem for people to want 
to teach a class how to convert itself to and from JSON.

Altering the API for dicts is a major step, so you would need to take this to 
python-ideas to start getting buy-in.   A much smaller API change would be to 
just teach the JSON module to recognize a __json__ method.

Presumably if a robust serialization solution is created, people will need a 
way to deserialize back into a named tuple, data class, or custom class.  
Offhand, the only way I can think of to do this would be to add a field that 
could be recognized by json.load().  Some care would be needed to not create a 
pickle-like risk of arbitrary code execution.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40077] Convert static types to heap types: use PyType_FromSpec()

[miss-islington]

miss-islington  added the comment:


New changeset bf108bb21e1d75e30bd17141cc531dd08a5e5d0c by Erlend Egeberg 
Aasland in branch 'master':
bpo-40077: Fix typo in simplequeue_get_state_by_type() (GH-23975)
https://github.com/python/cpython/commit/bf108bb21e1d75e30bd17141cc531dd08a5e5d0c


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42754] Unpacking of literals inside other literals should be optimised away by the compiler

[Batuhan Taskaya]

Change by Batuhan Taskaya :


--
nosy: +BTaskaya

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42768] super().__new__() of list expands arguments

[Richard Neumann]

Richard Neumann  added the comment:

I could have sworn, that this worked before, but it was obviously me being 
tired at the end of the work day.
Thanks for pointing this out and sorry for the noise.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30963] xxlimited.c XxoObject_Check should be XxoObject_CheckExact

[hai shi]

hai shi  added the comment:

> #define XxoObject_CheckExact(v) (Py_TYPE(v) == _Type)
`#define Xxo_CheckExact(obj) Py_IS_TYPE(obj, _Type)` would be better to 
hide details now.

> #define XxoObject_Check(v) PyObject_TypeCheck(v, _Type)
+1

I think the annotation will worth to be updaetd.
https://github.com/python/cpython/blob/master/Modules/xxlimited.c#L76

--
keywords: +easy
nosy: +petr.viktorin, shihai1991, vstinner
versions: +Python 3.10 -Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40810] sqlite3 test CheckTraceCallbackContent fails for sqlite v3.7.3 through 3.7.14.1

[Dong-hee Na]

Change by Dong-hee Na :


--
nosy: +corona10

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42767] Review usage of atomic variables in signamodule.c

[hai shi]

Change by hai shi :


--
nosy: +shihai1991

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42768] super().__new__() of list expands arguments

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

Your problem is with list.__init__ method. It expects at most one argument.

tuple does not have specialized __init__ method, it inherits it from object. 
All work is done in tuple.__new__. list does not have specialized __new__ 
method, it inherits it from object. All work is done in list.__init__.

If your override __new__ with incompatible signature, make also __init__ 
supporting it.

--
nosy: +serhiy.storchaka
resolution:  -> not a bug
stage:  -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42068] For macOS, package the included Tcl and Tk frameworks in a rational way.

[Marc Culler]

Marc Culler  added the comment:

Hi Ned, I have a comment about the code signing issues that would arise if the 
Tcl and Tk frameworks were embedded as actual subframeworks inside the 
Python.framework, and a user later replaced those with newer versions. The 
answer is that no new issues arise.  The current Python.framework fails 
codesign verification as soon as a single new package is installed with pip 
(even upgrading pip itself).  The codesign command detects the new files and 
exits with non-zero status if called with --verify.  In verbose mode it will 
list all of the files that were added.

This does not prevent python from working.  So it should just be acknowledged 
that python's codesigning is used purely to make installation simpler and that 
no attempt is currently being made to ensure that an installed python framework 
continues to pass codesign verification.  As a consequence, my proposal in this 
ticket would raise no new codesigning issues, but it would allow users to much 
more easily upgrade the Tcl and Tk embedded in 
/Library/Frameworks/Python.framework versions.  It would also simplify the 
recipe for building Tcl and Tk in the buildscript.

While the python framework may be viewed as a black box, in fact it is not a 
black box at all.  It can be changed at will either by python itself or by a 
user.  Making its structure clearer and cleaner could only be an improvement.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42769] concurrent.futures.ProcessPoolExecutor is unable to forward exceptions with state.

[Damien Levac]

New submission from Damien Levac :

When running tasks on a `ProcessPoolExecutor`, exceptions raised by the 
dispatched function should be pickled and accessible to the parent process 
through the `Future.exception` method.

On Python 3.9.1 (Linux ryzen3950x 5.4.0-58-generic #64-Ubuntu SMP Wed Dec 9 
08:16:25 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux) the behavior works with 
exceptions which are stateless but not if they hold state.

I suspect it is related to the multiprocessing/pickle bug mentioned in the 
release notes to 3.9.1 but I didn't dig much deeper.

Let me know if I can assist in any way or if any pertinent information is 
missing: it is my first time reporting a bug here :)

Thank you for your hard work!

--
components: Library (Lib)
files: repro.py
messages: 383903
nosy: damien.levac
priority: normal
severity: normal
status: open
title: concurrent.futures.ProcessPoolExecutor is unable to forward exceptions 
with state.
type: behavior
versions: Python 3.9
Added file: https://bugs.python.org/file49703/repro.py

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42768] super().__new__() of list expands arguments

[Richard Neumann]

New submission from Richard Neumann :

When sublassing the built-in list, the invocation of super().__new__ will 
unexpectedly expand the passed arguments:

class MyTuple(tuple):

def __new__(cls, *items):
print(cls, items)
return super().__new__(cls, items)


class MyList(list):

def __new__(cls, *items):
print(cls, items)
return super().__new__(cls, items)


def main():

my_tuple = MyTuple(1, 2, 3, 'foo', 'bar')
print('My tuple:', my_tuple)
my_list = MyList(1, 2, 3, 'foo', 'bar')
print('My list:', my_list)


if __name__ == '__main__':
main()


Actual result:

 (1, 2, 3, 'foo', 'bar')
My tuple: (1, 2, 3, 'foo', 'bar')
 (1, 2, 3, 'foo', 'bar')
Traceback (most recent call last):
  File "/home/neumann/listbug.py", line 24, in 
main()
  File "/home/neumann/listbug.py", line 19, in main
my_list = MyList(1, 2, 3, 'foo', 'bar')
TypeError: list expected at most 1 argument, got 5


Expected:

 (1, 2, 3, 'foo', 'bar')
My tuple: (1, 2, 3, 'foo', 'bar')
 (1, 2, 3, 'foo', 'bar')
My list: [1, 2, 3, 'foo', 'bar']

--
components: ctypes
messages: 383902
nosy: conqp
priority: normal
severity: normal
status: open
title: super().__new__() of list expands arguments
type: behavior
versions: Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42767] Review usage of atomic variables in signamodule.c

[Dong-hee Na]

Change by Dong-hee Na :


--
nosy: +corona10

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42767] Review usage of atomic variables in signamodule.c

[STINNER Victor]

New submission from STINNER Victor :

In bpo-41713, I ported the _signal module to the multi-phase initialization 
API. I tried to move the signal state into a structure to cleanup the code, but 
I was scared by the following atomic variables:
---
static volatile struct {
_Py_atomic_int tripped;
PyObject *func;
} Handlers[NSIG];

#ifdef MS_WINDOWS
#define INVALID_FD ((SOCKET_T)-1)

static volatile struct {
SOCKET_T fd;
int warn_on_full_buffer;
int use_send;
} wakeup = {.fd = INVALID_FD, .warn_on_full_buffer = 1, .use_send = 0};
#else
#define INVALID_FD (-1)
static volatile struct {
#ifdef __VXWORKS__
int fd;
#else
sig_atomic_t fd;
#endif
int warn_on_full_buffer;
} wakeup = {.fd = INVALID_FD, .warn_on_full_buffer = 1};
#endif

/* Speed up sigcheck() when none tripped */
static _Py_atomic_int is_tripped;
---

For me, the most surprising part is Handlers[signum].tripped which is declared 
as volatile *and* an atomic variable.

I'm not sure if Handlers[signum].func must be volatile neither.

Also, wakeup.fd is declared as sig_atomic_t on Unix. Could we use an atomic 
variable instead, or is it important to use "sig_atomic_t"?

--

I recently added pycore_atomic_funcs.h which provides functions to access 
variables atomically. It uses atomic functions if available, or falls back on 
"volatile" otherwise. Maybe this approach would be interesting here, maybe for 
Handlers[signum].func?

--
components: Interpreter Core
messages: 383901
nosy: vstinner
priority: normal
severity: normal
status: open
title: Review usage of atomic variables in signamodule.c
versions: Python 3.10

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40522] [subinterpreters] Get the current Python interpreter state from Thread Local Storage (autoTSSkey)

[STINNER Victor]

STINNER Victor  added the comment:

PR 23976 should make _PyInterpreterState_GET() more efficient, and it prepares 
the code base for one GIL per interpreter (which is purpose of this issue ;-)).

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40522] [subinterpreters] Get the current Python interpreter state from Thread Local Storage (autoTSSkey)

[STINNER Victor]

STINNER Victor  added the comment:

PR 23976 stores the currrent interpreter and the current Python thread state 
into a Thread Local Storage (TLS) using GCC/clang __thread keyword.

On x86-64 using LTO and GCC -O3, _PyThreadState_GET() and 
_PyInterpreterState_GET() become a single MOV.

Assembly code using LTO and gcc -O3.

_PyThreadState_GET() in _PySys_GetObjectId():

   0x004adabe <+14>: mov rbx,QWORD PTR fs:0xfff8

_PyThreadState_GET() in PyThreadState_Get():

   0x0046b660 <+0>: mov  rax,QWORD PTR fs:0xfff8

_PyInterpreterState_GET() in PyTuple_New():

   0x0048dfcc <+12>: mov  rax,QWORD PTR fs:0xfff0

_PyInterpreterState_GET() in PyState_FindModule():

   0x0044bf20 <+16>: mov  rax,QWORD PTR fs:0xfff0

---

Note: Without LTO, sometimes there is an indirection:

_PyThreadState_GET() in _PySys_GetObjectId(), 2 MOV (PIC indirection):

   movrax,QWORD PTR [rip+0x1eb270]# 0x713fe0
   # rax = 0xfff0 (-16)
   movr13,QWORD PTR fs:[rax]

_PyInterpreterState_GET() in PyTuple_New(), 2 MOV (PIC indirection):

  movrax,QWORD PTR [rip+0x294d95]# 0x713ff8
  movrax,QWORD PTR fs:[rax]

An optimized Python should always be built with LTO.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42766] urllib.request.HTTPPasswordMgr uses commonprefix instead of commonpath

[Donát Nagy]

New submission from Donát Nagy :

The is_suburi(self, base, test) method of HTTPPasswordMgr in the urllib.request 
module tries to "Check if test is below base in a URI tree", but it uses the 
posixpath.commonprefix() function. This is problematic because commonprefix 
ignores the path structure (for example commonprefix(['/usr/lib', 
'/usr/local/lib'])=='/usr/l') and therefore the current implementation of 
is_suburi is essentially equivalent to calling str.startswith after some 
normalization steps.

If we want to say that example.com/resource101 is *NOT* below 
example.com/resource1 in a URI tree, then the call to commonprefix should be 
replaced by a call to posixpath.commonpath(), which does the right thing.

--
components: Library (Lib)
messages: 383898
nosy: nagdon
priority: normal
severity: normal
status: open
title: urllib.request.HTTPPasswordMgr uses commonprefix instead of commonpath
type: behavior
versions: Python 3.10

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42765] Introduce new data model method __iter_items__

[Karthikeyan Singaravelan]

Change by Karthikeyan Singaravelan :


--
nosy: +rhettinger

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40522] [subinterpreters] Get the current Python interpreter state from Thread Local Storage (autoTSSkey)

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +22821
pull_request: https://github.com/python/cpython/pull/23976

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com