Guido van Rossum added the comment:
New changeset 131d5516409791b170b09a6ef8ed8463c9b09015 by db3l in branch
'master':
bpo-43271: Re-enable ceval.c optimizations for Windows debug builds (GH-24739)
https://github.com/python/cpython/commit/131d5516409791b170b09a6ef8ed84
Guido van Rossum added the comment:
Thanks for all your research and the PR!
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Guido van Rossum added the comment:
Maybe we need to add an API (e.g. a flag to compile()) so that we can ask the
actual parser the question we're interested in, rather than having to use hacks
and heuristics?
--
___
Python tracker
&
Guido van Rossum added the comment:
SGTM
--
___
Python tracker
<https://bugs.python.org/issue29982>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
Maybe return the original string?
--
___
Python tracker
<https://bugs.python.org/issue43463>
___
___
Python-bugs-list mailin
Change by Guido van Rossum :
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue27501>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
It works just fine with http:// (redirects to https).
Thanks for testing our workflow. It works just fine.
--
resolution: -> not a bug
stage: patch review -> resolved
status: open -> closed
___
Pytho
Guido van Rossum added the comment:
- Double forward ref: IMO this can be resolved in the get_type_hints()
functions. (Łukasz do you agree?)
- inspect.signature(): Maybe this could switch to using
typing.get_type_hints()? Then again if performance is important here maybe we
cannot change
Guido van Rossum added the comment:
It,sounds like you are trying to get the effect of putting 'from __future__
import dataclasses' at the top of dataclasses'.py, right? Are you saying that
without this, the output of the test sc
Guido van Rossum added the comment:
Have you come up with a fix yet? (Preferably a fix that can be applied locally
to dataclasses.py.)
--
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
To clarify, for Eric: what Batuhan did is effectively adding `from __future__
import annotations` to the top of dataclasses.py.
I believe the root cause is that dataclasses creates functions by synthesizing
`def` statements (in _create_fn()) and the
Guido van Rossum added the comment:
You can do this as part of your mega-PR to turn on `from __future__ import
annotations` by default.
--
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
I think in general it is more insightful to discuss the behavior of
get_type_hints() given specific things in annotations.
We generally don't write forward refs inside forward refs, like
"SomeClass['int']". So maybe that code was
Guido van Rossum added the comment:
There will still be a lot of code written that way, because people need
compatibility with earlier versions of Python. So I think it should be fixed in
get_type_hints().
--
___
Python tracker
<ht
Guido van Rossum added the comment:
Serhiy, what do you mean by "otherwise we could run out of file descriptiors"?
I looked a bit at the code and there are different kinds of algorithms involved
for different forms of patterns, and the code also takes vastly different paths
for
Guido van Rossum added the comment:
Hm, yeah.
Perhaps we can add some buffering so that for directories with a small
number of files (say < 1000) the FD is closed before recursing into it,
while for large directories it keeps the FD open until the last block of
1000 DirEntries is read? I
Guido van Rossum added the comment:
I hope some volunteer will submit a doc PR.
In the meantime, throwing out one more idea: perhaps my first idea, to make
_glob1() an iterator, could work if we only do it for leaves of the
pattern, so for the a*/b*/c* example, only for the c* part
Guido van Rossum added the comment:
Sure looks like a tokenizer issue to me. For example this is broken in both
versions:
pass
\
pass
It complains about an unexpected indent, but it should really be considered a
blank line broken in two -- a backslash is supposed to just erase itself
Guido van Rossum added the comment:
So does that make this "not a bug"? Or is there something to document? For
technical reasons we can't just add a __init__ method to Generic, and I doubt
that it's feasible to chan
Change by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<https://bugs.python.org/issue7105>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
I think this can wait until after beta 2.
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
Changing the topic to not point fingers at Generic.
--
nosy: -levkivskyi
title: Inheriting from Generic causes inspect.signature to always return
(*args, **kwargs) for constructor (and all subclasses) -> Inheriting from class
that defines __ne
Guido van Rossum added the comment:
To be clear, I consider it a bug.
--
___
Python tracker
<https://bugs.python.org/issue40847>
___
___
Python-bugs-list mailin
Guido van Rossum added the comment:
Honestly I see no reason to keep that easter egg. Can we remove it please?
--
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
New changeset bcd7deed9118e365c1225de2a2e1a81bf988c6ab by Lysandros Nikolaou in
branch 'master':
bpo-40939: Remove PEG parser easter egg (__new_parser__) (#20802)
https://github.com/python/cpython/commit/bcd7deed9118e365c1225de2a2e1a8
Change by Guido van Rossum :
--
resolution: -> fixed
status: open -> closed
versions: +Python 3.9
___
Python tracker
<https://bugs.python.org/i
Guido van Rossum added the comment:
Yes, it's clear. :-) This is one reason we need to migrate off bpo and onto
GitHub.
--
___
Python tracker
<https://bugs.python.org/is
Guido van Rossum added the comment:
How's this going?
--
___
Python tracker
<https://bugs.python.org/issue22167>
___
___
Python-bugs-list mailing list
Guido van Rossum added the comment:
Sounds good.
FWIW, and totally off-topic, I find it annoying that pathlib's .glob() method
supports ** in patterns, but its cousing .match() does not:
>>> p = pathlib.Path("Lib/test/support/os_helper.py")
>>
Guido van Rossum added the comment:
And here I had hoped that this was a Pull Request. :-)
Note that the docs also still state that these are ABCs -- in fact they are
Protocols.
I'm not sure why the same info should be repeated for the `@runtime_checkable`
decorator t
Change by Guido van Rossum :
--
nosy: +gvanrossum
nosy_count: 5.0 -> 6.0
pull_requests: +20120
pull_request: https://github.com/python/cpython/pull/20921
___
Python tracker
<https://bugs.python.org/issu
Guido van Rossum added the comment:
New changeset 59cf853332a82ce92875ea3dd6bba08e1305a288 by Ram Rachum in branch
'master':
bpo-40636: Documentation for zip-strict (#20961)
https://github.com/python/cpython/commit/59cf853332a82ce92875ea3dd6bba0
Guido van Rossum added the comment:
Every other detail about f-strings is spelled out meticulously in the reference
manual:
https://docs.python.org/3/reference/lexical_analysis.html#formatted-string-literals
I think it needs to be added here.
PS OT: I can't find anything about f-strin
Guido van Rossum added the comment:
You can delete symbol.py -- it has no use now that the old parser is gone.
We should probably also update the regeneration targets in the Makefile. (At
least review them.)
--
___
Python tracker
<ht
Guido van Rossum added the comment:
But it's a bug, right? The intention for sure is that the cf_feature_version
field is only used when the PyCF_ONLY_AST flags is set in cf_flags, per the
docs you cite as [3]. Why wouldn't it be possible to fix that?
--
status: clos
Guido van Rossum added the comment:
Good catch!
Are all the coverity complaints about this kind of code?
if (_res == NULL && PyErr_Occurred()) {
...
}
(Mostly for Pablo and Lysandros:) This comes from emit_action(). It is preceded
by a line of the form
_res = ;
Most of the
Change by Guido van Rossum :
--
nosy: +gvanrossum, pablogsal
___
Python tracker
<https://bugs.python.org/issue41060>
___
___
Python-bugs-list mailing list
Unsub
Change by Guido van Rossum :
--
pull_requests: +20193
pull_request: https://github.com/python/cpython/pull/21021
___
Python tracker
<https://bugs.python.org/issue35
Guido van Rossum added the comment:
I'm fixing it -- can you verify? I have no easy way to test this. DO you think
we should also add a note to the docs that this was broken in 3.8.0-3?
--
___
Python tracker
<https://bugs.python.org/is
Change by Guido van Rossum :
--
status: closed -> open
___
Python tracker
<https://bugs.python.org/issue35975>
___
___
Python-bugs-list mailing list
Unsubscrib
Guido van Rossum added the comment:
Hm, the backports have to be done manually, since the 3.8 parser and the 3.10
parser don't overlap, and 3.9 has both versions...
--
___
Python tracker
<https://bugs.python.org/is
Change by Guido van Rossum :
--
pull_requests: +20194
stage: resolved -> patch review
pull_request: https://github.com/python/cpython/pull/21022
___
Python tracker
<https://bugs.python.org/issu
Change by Guido van Rossum :
--
pull_requests: +20195
pull_request: https://github.com/python/cpython/pull/21023
___
Python tracker
<https://bugs.python.org/issue35
Guido van Rossum added the comment:
See also test_carloverre() in test_descr.py. I don't recall exactly what this
was about, but that test is definitely relevant.
--
___
Python tracker
<https://bugs.python.org/is
Guido van Rossum added the comment:
Okay, I will add that diff (I'll probably add async def to the set of features
tested). Do you mind if I just incorporate it in my diff? I could add an
acknowledgment in the commit message. Otherwise you could send a separa
Guido van Rossum added the comment:
I don't think we should deprecate List etc. (PEP 585 specifically punts on the
topic) but it's good to start mentioning list[t] etc. in the docs. Do you want
to volunteer to write some docs?
--
Guido van Rossum added the comment:
That sounds like the right thing to do. You should be able to recognize pure
Python __setattr__ implementations by the presence of slot_tp_setattro in the
tp_setattro slot. (Likewise for __delattr__.)
It's been really long since I looked at this --
Guido van Rossum added the comment:
But it is undocumented and doesn’t work with -X oldparser.
--
--Guido (mobile)
--
___
Python tracker
<https://bugs.python.org/issue12
Guido van Rossum added the comment:
New changeset 9d197c7d48147a9ea2f7f7be917f35514a16524b by Guido van Rossum in
branch 'master':
bpo-35975: Only use cf_feature_version if PyCF_ONLY_AST in cf_flags (#21021)
https://github.com/python/cpython/commit/9d197c7d48147a9ea2f7f7be917f35
Guido van Rossum added the comment:
New changeset 2a1ee1d970460047bd88da9638f8c1789431d9ab by Guido van Rossum in
branch '3.9':
[3.9] bpo-35975: Only use cf_feature_version if PyCF_ONLY_AST in cf_flags
(#21022)
https://github.com/python/cpyt
Guido van Rossum added the comment:
New changeset e653369e76d7da6bcbcf1f09a141f47fb77df6c3 by Guido van Rossum in
branch '3.8':
[3.8] bpo-35975: Only use cf_feature_version if PyCF_ONLY_AST in cf_flags
(#21023)
https://github.com/python/cpyt
Guido van Rossum added the comment:
Fixed again. Thanks Stefan for finding this! I apologize for the bug.
--
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/i
Guido van Rossum added the comment:
The IDLE feature should not change, for all the reasons Terry have.
I am quite done with changes to PEP in order to settle arguments, and I do not
believe this convention needs mentioning in that PEP. Remember there is text in
PEP 8 reminding readers to
Guido van Rossum added the comment:
Stefan can you do the 3.8 backport?
--
___
Python tracker
<https://bugs.python.org/issue39960>
___
___
Python-bugs-list m
Change by Guido van Rossum :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Guido van Rossum added the comment:
Should this be backported? How far back?
--
___
Python tracker
<https://bugs.python.org/issue39168>
___
___
Python-bug
Change by Guido van Rossum :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Guido van Rossum added the comment:
You're welcome. Hope this helps Cpython users.
--
___
Python tracker
<https://bugs.python.org/issue39960>
___
___
Pytho
Guido van Rossum added the comment:
There's no python-dev discussion; if you want more feedback I recommend
starting on python-ideas first (on either forum you may expect pushback because
this is not about a proposed change to Python or its workflow).
The Lib/ast.py module will contin
Guido van Rossum added the comment:
Can that be done as a 3rd party wrapper? Then you would be able to support
older Python versions, and typed_ast (which can parse older Python grammars
with a newer Python that's older than 3.8). Plus it would be much easier to get
your code released
Change by Guido van Rossum :
--
nosy: +gvanrossum, levkivskyi
___
Python tracker
<https://bugs.python.org/issue41249>
___
___
Python-bugs-list mailing list
Unsub
Guido van Rossum added the comment:
Can you send a PR against what’s new 3.8?
On Fri, Jul 10, 2020 at 20:14 JustAnotherArchivist
wrote:
>
> JustAnotherArchivist added the comment:
>
> As another datapoint, this also broke some of my code on 3.8 because I
Guido van Rossum added the comment:
Thanks for joining in! How do you do incremental parsing with LL1 currently?
FWIW I found
https://ohmlang.github.io/pubs/sle2017/incremental-packrat-parsing.pdf which
may have some useful ideas.
--
___
Python
Guido van Rossum added the comment:
Reopening because there appears to be a problem with the fix (see PR 21473).
--
status: closed -> open
___
Python tracker
<https://bugs.python.org/issu
Guido van Rossum added the comment:
@Stefan Do you agree that the fix proposed in PR 21473 needs to be made?
--
___
Python tracker
<https://bugs.python.org/issue39
Guido van Rossum added the comment:
The PEP now says 3.10, so the docs for __future__ should be fixed.
--
resolution: not a bug ->
status: closed -> open
___
Python tracker
<https://bugs.python.org/i
Guido van Rossum added the comment:
The 3.9 and 3.8 backports are waiting for tests to complete. The 3.7 and 3.6
backports need to be merged by the RM (Ned). Then someone can close this issue.
--
nosy: +gvanrossum
___
Python tracker
<ht
Guido van Rossum added the comment:
We have a buildbot failure: test_asyncio altered the execution environment.
What does that mean? Victor?
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue41
Guido van Rossum added the comment:
@Łukasz: this should be good for the 3.8.5 release.
--
___
Python tracker
<https://bugs.python.org/issue41295>
___
___
Pytho
Guido van Rossum added the comment:
> It should also include 0x20 (space) since that can also be used to manipulate
> the request.
Can you indicate how to use a space in the HTTP verb as part of an attack?
--
___
Python tracker
Guido van Rossum added the comment:
New changeset 653f420b53a3aa87316cef59de8d3f5d9e11deb4 by wyfo in branch
'master':
bpo-41341: Recursive evaluation of ForwardRef in get_type_hints (#21553)
https://github.com/python/cpython/commit/653f420b53a3aa87316cef59de8d3f
Guido van Rossum added the comment:
Łukasz, can I please have a decision on whether to backport this bugfix to 3.9?
See my comment about that:
https://github.com/python/cpython/pull/21553#pullrequestreview-452895735
--
___
Python tracker
<ht
Guido van Rossum added the comment:
I think mentioning this in the docs is the best we can do in 3.9, and for 3.10
the point will be moot. The next release is release candidate 1, so we're well
past the point where we can implement new function
Guido van Rossum added the comment:
I've reviewed your first PR (#21509).
Regarding your second one, I think there's a misunderstanding. With "pydoc" I
didn't mean "the Python docs". IMO the documentation of f-strings in the
language reference is suff
Guido van Rossum added the comment:
This looks like a useful addition. I hope someone will take up the review!
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
Łukasz, what do you think of backporting this to 3.9?
--
nosy: +gvanrossum, lukasz.langa
___
Python tracker
<https://bugs.python.org/issue41
Guido van Rossum added the comment:
Why not add Travis-CI and AppVeyor?
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue41328>
___
___
Guido van Rossum added the comment:
Either remove it or close this issue as won’t fix.
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue41
Guido van Rossum added the comment:
This surely is some application corner case. Closing.
--
nosy: +gvanrossum
resolution: -> not a bug
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Guido van Rossum added the comment:
Is this really worth our attention? In 3.10 this will be irrelevant.
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue41
Guido van Rossum added the comment:
We should not do this, it would expose internals that we need to keep private.
The right approach would be to keep things as bytes.
--
nosy: +gvanrossum
resolution: -> wont fix
stage: -> resolved
status: open -&g
Guido van Rossum added the comment:
There already is a decent set of examples. I think a Snake game is a worthy
addition.
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue41
Guido van Rossum added the comment:
Waiting for someone to submit a PR.
--
___
Python tracker
<https://bugs.python.org/issue41314>
___
___
Python-bugs-list m
Guido van Rossum added the comment:
Marking this as "easy". People are welcome to submit PRs that fix the wording
in one or a small number of modules called out in Serhiy's list.
--
keywords: +easy
nosy: +gvanrossum
___
Python
Guido van Rossum added the comment:
We might get more help with this issue if the process of updating the help()
output were better documented. I recently looked into what it would take to add
"FSTRINGS" to the "topics" list, and it's quite complicated -- I only
Guido van Rossum added the comment:
Sorry, I realize what's wrong here. When we switched to the new PEG parser (the
"pegen" you saw mentioned in the warning) we switched to a new grammar file.
You have to change Grammar/python.gram instead of Grammar/Grammar.
We couldn
Guido van Rossum added the comment:
There are some difficulties with removing Grammar/Grammar, since it is used to
generate the full grammar in the reference docs (Doc/reference/grammar.rst).
Producing a similar grammar from the PEG grammar is currently painful because
our PEG grammar
Guido van Rossum added the comment:
@ned This will need your assistance to merge the 3.7 backport IIUC.
--
nosy: +ned.deily
___
Python tracker
<https://bugs.python.org/issue41
Guido van Rossum added the comment:
Never mind, Miss Islington failed to backport to 3.7, so let's forget about
that.
--
versions: -Python 3.7
___
Python tracker
<https://bugs.python.org/is
Guido van Rossum added the comment:
This can be closed. It needs docs though; there's a separate issue for that
(issue40814).
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python t
Guido van Rossum added the comment:
Alternatively, we at least need a section of documentation for things like
`list[int]` and `dict[str, float]`.
--
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
Hm, recursive type aliases are an interesting issue. We may be able to do
better there for 3.10, even if we can't fix it for 3.9 (or at least not for
3.9.0).
But in the meantime maybe you can come up with a PR that adds a note to the
typing docs in
Guido van Rossum added the comment:
In general, "4.0" means "never", and for some deprecations that seems totally
appropriate. I wouldn't worry about it. I'm closing this issue now that all
three PRs have been merged. Thanks for your contribution!
Guido van Rossum added the comment:
New changeset f117cef25b5ffc4db9fbe373ddb65e14f59f0397 by Shankar Jha in branch
'master':
bpo-39868: Add documentation for Assignment Expressions (walrus, PEP 572)
(#18851)
https://github.com/python/cpython/commit/f117cef25b5ffc4db9fbe373ddb65e
Guido van Rossum added the comment:
Congratulations SHANKAR JHA. Thank you for your contribution.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.10, Python 3.9
___
Python tracke
Change by Guido van Rossum :
--
pull_requests: +20765
pull_request: https://github.com/python/cpython/pull/21624
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
There’s probably also something in configure.in that can switch this on.
--
___
Python tracker
<https://bugs.python.org/issue41
Change by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<https://bugs.python.org/issue41340>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<https://bugs.python.org/issue41380>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
Thank you Joseph Perez! Looking forward to more of your contributions.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
2801 - 2900 of 5533 matches
Mail list logo