Václav Šmilauer added the comment:
Ping? Roumen has been updaing the patch for one year and no reaction. Is there
something which can be done?
--
nosy: +eudoxos
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18654
Serhiy Storchaka added the comment:
Doesn't always zeroes == length?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24000
___
___
Jaivish Kothari added the comment:
Please find the attached patch for review.
--
keywords: +patch
resolution: - fixed
Added file: http://bugs.python.org/file39118/doc_patch.patch
___
Python tracker rep...@bugs.python.org
Christian Heimes added the comment:
You could be right. I didn't track all paths manually. All this bit shifting is
making my head dizzy... :)
Anyways I have sent you an invite for Coverity, so you can check the result
yourself. The Python test suite passes with assert(k 32); inside the
Larry Hastings added the comment:
p.s. I'm now leaning heavily towards renaming types to accept, and putting
NoneType in the set passed to accept= instead of a separate parameter.
--
___
Python tracker rep...@bugs.python.org
Mark Dickinson added the comment:
Ah, sorry; I see it. Fix on the way.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23999
___
___
Serhiy Storchaka added the comment:
Converters with encoding=True are not used in converted code. es is never
used in still non-converted code and et is used only 6 times (for idna and
utf-8 encodings) and can be replaced with custom converter or inlined code.
So I think the support of
Roundup Robot added the comment:
New changeset 558335559383 by doko in branch 'default':
- #22980: fix triplet configure test for more targets
https://hg.python.org/cpython/rev/558335559383
--
___
Python tracker rep...@bugs.python.org
Matthias Klose added the comment:
I'm not trying to discredit any use cases, I just don't see them.
so why do you see this on x86 for 32/64bit, but not for ARM
soft-float/hard-float. The example given was pretty clear.
All Linux distributions I know place the 32-bit and 64-bit versions
of
Larry Hastings added the comment:
Actually eval(ast) is all I need for #23967 too. But eval is a builtin, so it
feels wrong to have it supporting--and therefore dependent on--ast.
--
___
Python tracker rep...@bugs.python.org
Larry Hastings added the comment:
Sadly, for political reasons, it's best that we not convert collections,
itertools, or random for now. I'll update this issue if the situation changes.
Sorry about that!
--
___
Python tracker
Serhiy Storchaka added the comment:
I think there is standard way to transform ast to bytecode and evaluate it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24002
___
Serhiy Storchaka added the comment:
I don't think that we should document such minor details in Tkinter
documentation. There are larger differences between Tcl/Tk versions that can
make Python programs fail (e.g. introducing new internal Tcl/Tk type), and they
are not documented.
--
Steven D'Aprano added the comment:
This is not a bug. while is a keyword, it is part of Python's syntax, and you
are not permitted to use keywords as variable names. This is not an accident,
but a deliberate decision.
--
nosy: +steven.daprano
resolution: - not a bug
status: open -
New submission from Jaivish Kothari:
https://docs.python.org/2/whatsnew/2.4.html?highlight=decorators#pep-318-decorators-for-functions-and-methods
'''
def require_int (func):
def wrapper (arg):
assert isinstance(arg, int)
return func(arg)
return wrapper
'''
New line is
Stefan Behnel added the comment:
Here's a better patch that avoids exception normalisation in all normal cases.
--
Added file: http://bugs.python.org/file39116/fix_stopiteration_crash.patch
___
Python tracker rep...@bugs.python.org
Larry Hastings added the comment:
I've posted about this to python-dev.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23920
___
___
Mark Dickinson added the comment:
Looking more closely, the report doesn't make sense to me: `k` is the return
value from a call to `lo0bits`. From the source of `lo0bits`, I don't see any
way that `k` can be 32: it's always going to be in the range [0, 31].
Christian: do you have any more
Mark Dickinson added the comment:
Okay, so after looking more closely, this *still* looks like a false positive:
`lo0bits` *can* return 32, but only for an input of zero. In the code in
question, we're doing `k = lo0bits(y)`, so the only way we can get a `k` of
`32` is if `y = 0`. But the
Mark Dickinson added the comment:
saw no output
Bah; missed a bit. I saw no output when running the Python test suite, that
is. That's not definitive, of course.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23999
Martin Falatic added the comment:
No, I don't expect something like the color change to be documented here
(unless that thing is incorrectly documented within python.org's current
release trees).
I do expect that just as python.org's OSX releases document the recommended
version of TclTk to
Martin Panter added the comment:
Sometimes the doc string for the overridden method does not make much sense in
the context of the subclass. Just wondering if this was considered; it seems
like a fairly serious downside to this new feature. E.g. in a package I am
reviewing, there is a class
Larry Hastings added the comment:
Oh, heavens, yes, that's much nicer. Thanks for the suggestion, Serhiy!
--
Added file:
http://bugs.python.org/file39115/larry.one.more.clinic.format.unit.map.cleanup.2.txt
___
Python tracker rep...@bugs.python.org
New submission from john kaiser:
found error when naming variables with basic functions
how to replicate
while=123 #while should be treated as variable name
while True: #this should be treated as a function
print while #this should be as a variable name
result:
File C:\Users\_you
New submission from Stefan Behnel:
asyncio/coroutines.py contains this code:
_COROUTINE_TYPES = (types.GeneratorType, CoroWrapper)
def iscoroutine(obj):
Return True if obj is a coroutine object.
return isinstance(obj, _COROUTINE_TYPES)
In other places, it uses inspect.isgenerator()
Georg Brandl added the comment:
Why is this a bug?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24005
___
___
Python-bugs-list mailing list
Antti Haapala added the comment:
This is *still* there in Hg tip: PyCFunction_Call in Objects/methodobject.c
does not have a case for `METH_KEYWORDS` only at all.
The documentation for METH_KEYWORDS says that it is *typically* coupled with
METH_VARARGS; however not having the case means that
Serhiy Storchaka added the comment:
For issue24001 you need rather eval(ast). But a function for stringifying ast
would be useful in any case.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24002
Mark Dickinson added the comment:
I'm pretty sure that our code was based on something rather more recent than
2001: it was the most recent version available at the time (around 2008?), and
it incorporates subsequent fixes from David Gay.
Please don't replace our dtoa.c with a current
Changes by Serhiy Storchaka storch...@gmail.com:
--
status: closed - open
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15582
___
___
Martin Falatic added the comment:
Yes, the python.org releases specify the TclTK they should be used with, for
OSX:
https://www.python.org/download/mac/tcltk/
If there's another python.org bug report list let me know, but this still seems
to be the right place. Since Python.org's windows
Martin Falatic added the comment:
FYI, I've filed a bug with the TclTk people regarding their documentation.
http://core.tcl.tk/tk/tktview/2a02881e4c23634022d0ae40a14383d9baad9eb9
--
___
Python tracker rep...@bugs.python.org
Eric Snow added the comment:
Just to be clear, I'm still -1 on any of this. On the one hand, there's a risk
of backward-compatibility breakage (just as much a corner-case as the need
expressed in this issue). On the other hand, I'd actually push for
_PyObject_LookupSpecial to be fixed to
Mark Lawrence added the comment:
Is case sensitivity still an issue on Windows? I've tried searching but there
are so many issues referring to case sensitivity that I got swamped.
--
components: +Windows
nosy: +BreamoreBoy, steve.dower, tim.golden, zach.ware
Cyd Haselton added the comment:
Ryan,
Found the missing fix.
In ./Programs/python.c
#ifndef __ANDROID__
oldloc = _PyMem_RawStrdup(setlocale(LC_ALL, NULL));
if (!oldloc) {
fprintf(stderr, out of memory\n);
return 1;
}
Berker Peksag added the comment:
That document is 10 years old :) I don't think it's worth to change now. Also,
what's new documents shouldn't be used as a tutorial. There are many
tutorials about writing decorators on the internet.
(Thanks for the report and the patch, Jaivish)
--
Ryan Gonzalez added the comment:
On Sun, Apr 19, 2015 at 7:16 PM, Cyd Haselton rep...@bugs.python.org
wrote:
Cyd Haselton added the comment:
Ryan,
Found the missing fix.
In ./Programs/python.c
#ifndef __ANDROID__
oldloc = _PyMem_RawStrdup(setlocale(LC_ALL, NULL));
if
Cyd Haselton added the comment:
Patch for python.c that prevents segfault on Android
--
Added file: http://bugs.python.org/file39135/python.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23496
Carol Willing added the comment:
janonymous, Thanks for the contribution. I agree with George that this is not a
bug. The whitespace exists for developer readability and maintainability of
code. Here's a section from PEP8 about the use of whitespace
Serhiy Storchaka added the comment:
Doesn't always zeroes == length?
zeroes requires length, but length does not require zeroes. As it
happens all the format units supported by str always have both parameters
either True or False. But the Py_UNICODE converter accepts length for
format
Serhiy Storchaka added the comment:
Looks as this is a patch for different issue.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24001
___
___
R. David Murray added the comment:
Find someone from the packaging sig willing to review it, maybe? As in, get on
that mailing list and ask.
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18654
Changes by Claudiu Popa pcmantic...@gmail.com:
--
stage: - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23701
___
___
Ned Deily added the comment:
(... and is documented in The Python 3 and 2 Language Reference manuals:
https://docs.python.org/3/reference/lexical_analysis.html#keywords and
https://docs.python.org/2/reference/lexical_analysis.html#keywords)
--
nosy: +ned.deily
Eric Snow added the comment:
s/TypeError/RuntimeError/
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23990
___
___
Python-bugs-list mailing
Changes by Berker Peksag berker.pek...@gmail.com:
--
components: +Extension Modules, Interpreter Core
nosy: +berker.peksag
versions: +Python 3.5 -Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8706
Eric Snow added the comment:
Ionel Cristian Mărieș added the comment:
#1. It's specified/documented, therefore it's intended
The first thing a maintainer does is check the docs. This is a sensible thing
to do - as you cannot have all the details in your hear. The main question at
that
Changes by Ryan Gonzalez rym...@gmail.com:
Added file: http://bugs.python.org/file39134/kbox_fix.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23496
___
Dmitry Kazakov added the comment:
I'll do that tomorrow. The patch still needs a review though...
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22619
___
Martin Falatic added the comment:
Thank you. Before going down the road of revising PEP 101 (which appears to be
very non-trivial despite the simple (and certainly always present) data
involved), I'd like to know: is version information about the pre-compiled
Windows binaries (of which this
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
--
assignee: - rhettinger
nosy: +rhettinger
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23990
___
Larry Hastings added the comment:
I don't care about this, but the patch looks fine. If this really helps then
LGTM.
Please hold off checking this in until after 3.5.0 alpha 4 is released.
--
___
Python tracker rep...@bugs.python.org
Eric Snow added the comment:
What would be the right thing?
My suggestion of using a metaclass is actually not effective here because
__call__ has meaning for metaclasses. Otherwise you could have made __call__
more dynamic via a metaclass. Except that is another reason why my suggestion
Ionel Cristian Mărieș added the comment:
I want to address the four main points of criticism in fixing this issue, just
in case it's not clear why I think those lines of thought are wrong:
#1. It's specified/documented, therefore it's intended
The first thing a maintainer does is check the
Eric Snow added the comment:
Note that (in my mind, unfortunately) the pickle module looks up several
dunder methods on instances. That isn't quite the same thing since the
issue is about callable not triggering the descriptor protocol. However it
is closely related. I point this out because
Larry Hastings added the comment:
Whoops. Here's the revised patch.
--
Added file:
http://bugs.python.org/file39123/larry.improved.signature.expressions.2.txt
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23967
Larry Hastings added the comment:
Thanks to #24002 I now know how to write evalify_node properly. This patch is
now much better.
Note that I deliberately made the new function _eval_ast_expr() as a private
module-level routine. I need that same functionality in Argument Clinic too,
so if
New submission from Anand Reddy Pandikunta:
inspect.getsource(sys.excepthook)
Traceback (most recent call last):
File stdin, line 1, in module
File /usr/local/lib/python2.7/inspect.py, line 701, in getsource
lines, lnum = getsourcelines(object)
File
Larry Hastings added the comment:
u# and Z# allow null characters.
Not according to the documentation. 'u' explicitly says it does not allow NUL
characters. 'Z', 'u#', and 'Z#' all say they are variants of 'u' but never
mention that they might allow NUL characters.
--
Ethan Furman added the comment:
The right thing, using a meta-class, is to have the meta-class check if the
proxied object is callable, and if so, put in the __call__ function in the
class that is being created.
--
___
Python tracker
Larry Hastings added the comment:
Here's the right patch.
--
Added file: http://bugs.python.org/file39128/larry.clinic.use.raw.types.2.txt
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24001
Changes by Larry Hastings la...@hastings.org:
Removed file:
http://bugs.python.org/file39122/larry.one.more.clinic.format.unit.map.cleanup.2.txt
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24001
New submission from Christian Heimes:
The init function of the signal module fails to check for errors in a couple of
places. The patch replaces PyDict_SetItemString() calls with
PyModule_AddIntMacro() and error checks. An exception is unlikely so I'm OK
when the patch just lands in 3.4 and
Larry Hastings added the comment:
Whoops. I'll fix that.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24001
___
___
Python-bugs-list mailing
Steven D'Aprano added the comment:
This bug report seems to be completely based on a false premise. In the very
first message of this issue, Ionel says:
it return True even if __call__ is actually an descriptor that raise
AttributeError (clearly not callable at all).
but that is wrong. It
Serhiy Storchaka added the comment:
Not according to the documentation. 'u' explicitly says it does not allow
NUL characters. 'Z', 'u#', and 'Z#' all say they are variants of 'u' but
never mention that they might allow NUL characters.
I understand the note in u description as explicitly
Claudiu Popa added the comment:
The latest patch drops the symlink check, since it can be added later.
--
Added file: http://bugs.python.org/file39133/issue22080_2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22080
Larry Hastings added the comment:
Doesn't always zeroes == length?
zeroes requires length, but length does not require zeroes. As it
happens all the format units supported by str always have both parameters
either True or False. But the Py_UNICODE converter accepts length for format
units
R. David Murray added the comment:
There is also no reason to break currently working code, which turning this
into a syntax error would od.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23275
Serhiy Storchaka added the comment:
I disagree. My goal with Argument Clinic is that third-party developers will
use it to write extensions. And we don't know how many extension modules are
using es, es#, et, et#, s#, y#, z#, u#, and U#. I don't think we can remove
any of this
New submission from Serhiy Storchaka:
Proposed patch makes Argument Clinic to output format argument of PyArg_Parse,
PyArg_ParseTuple and PyArg_ParseTupleAndKeywords in a line with a function
itself. First, this makes generated code more compact and easier to read and
compare with old code.
Larry Hastings added the comment:
Thanks to #24002 I now know how to write evalify_node properly. This revision
of the patch is much better, and maybe ready for checkin.
--
Added file:
http://bugs.python.org/file39122/larry.one.more.clinic.format.unit.map.cleanup.2.txt
Changes by Zachary Ware zachary.w...@gmail.com:
--
stage: - resolved
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24003
___
___
Python-bugs-list
Larry Hastings added the comment:
New diff based on Serhiy's latest round of comments. Thanks, Serhiy! You are
inexhaustable!
--
Added file:
http://bugs.python.org/file39125/larry.one.more.clinic.format.unit.map.cleanup.3.txt
___
Python tracker
R. David Murray added the comment:
Thanks for the patch. We'll want a unit test for the behavior before
committing this.
--
nosy: +r.david.murray
stage: - test needed
versions: +Python 3.5
___
Python tracker rep...@bugs.python.org
New submission from Serhiy Storchaka:
There are a lot of format units supported in PyArg_Parse* functions, but some
of them are rarely or never used in current CPython code. Some of format units
are legacy from Python 2 and are not needed in modern Python 3 code or can be
replaced with custom
R. David Murray added the comment:
I'm guessing you are on Windows. Please read
https://docs.python.org/2/library/multiprocessing.html#windows. You can't do
what you show on windows (though you can on unix, since it uses fork).
--
nosy: +r.david.murray
resolution: - not a bug
New submission from Christian Heimes:
The init function of the locale module fails to check for errors in a couple of
places. The patch replaces PyDict_SetItemString() calls with
PyModule_AddIntMacro() and error checks. An exception is unlikely so I'm OK
when the patch just lands in 3.4 and
Serhiy Storchaka added the comment:
Needed a documentation. I'm not interested in writing it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22619
___
New submission from Ben Hoyt:
Victor Stinner's documentation for os.scandir and DirEntry is a great start
(https://docs.python.org/dev/library/os.html#os.scandir), however there are a
few mistakes in it, and a few ways I think it could be improved. Attaching a
patch with the following overall
New submission from ppperry:
An AttributeError is raised when starting a new process with an object defined
in the interactive interpreter
def test():print test
test()
test
from threading import Thread
Thread(target=test).start()
test
from multiprocessing import Process
Changes by R. David Murray rdmur...@bitdance.com:
--
stage: patch review - commit review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23725
___
Larry Hastings added the comment:
There is! compile() will do it, though the docstring doesn't mention it. (The
full documentation does.)
The following function meets both my use cases:
def eval_ast_expr(node, symbols=None, *, filename='-'):
Takes an ast.Expr node.
Carol Willing added the comment:
Berker, Thanks for the patch review. I'm going to work with the folks at Sphinx
project this week to see if the headings are fixed in a newer Sphinx version.
If so, I will make the change here. If not, I will follow your recommendation
to close and create an
Ionel Cristian Mărieș added the comment:
On Sun, Apr 19, 2015 at 9:01 PM, Eric Snow rep...@bugs.python.org wrote:
Finally, instead of changing callable, why not use a metaclass that does
the right thing? I believe MagicMock does something along those lines.
What would be the right thing?
Terry J. Reedy added the comment:
The tkinter docs need to be expanded, but that is a different issue.
The complete tcl/tk version is displayed in Idle - Help - About Idle using
self.tk.call('info', 'patchlevel'). Pending a reason not to, I would be in
favor of adding this full info as a
Changes by Riley Banks waul...@gmail.com:
--
nosy: +vaultah
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22619
___
___
Python-bugs-list mailing
Tim Golden added the comment:
One small thing to bear in mind is that the existing code (ie with the extra
linefeed) raises an IndentationError if cut-and-pasted into the interactive
interpreter; with the OP's change, it succeeds. Might not have been their
intention, but certainly is the
Changes by Claudiu Popa pcmantic...@gmail.com:
--
stage: needs patch - test needed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21574
___
___
New submission from Christian Heimes:
Similar to #24011 and #24010 the pyexpat module's init function fails to check
some return values for NULL. The patch doesn't include proper reference
cleanups as most of the other parts of PyInit_pyexpat() don't cleanup on error,
too.
CID 982779 (#2 of
STINNER Victor added the comment:
Related issues and discussions:
- [Python-Dev] open(): set the default encoding to 'utf-8' in Python 3.3?
https://mail.python.org/pipermail/python-dev/2011-June/112086.html
- Issue #12451: open: avoid the locale encoding when possible
Ionel Cristian Mărieș added the comment:
On Sun, Apr 19, 2015 at 10:01 PM, Ethan Furman rep...@bugs.python.org
wrote:
The right thing, using a meta-class, is to have the meta-class check if
the proxied object is callable, and if so, put in the __call__ function in
the class that is being
Claudiu Popa added the comment:
That's actually expected, since sys.excepthook is a builtin, so there's no
source code to be retrieved (the same holds for other builtins, such as next,
range etc).
--
nosy: +Claudiu.Popa
resolution: - not a bug
stage: - resolved
status: open - closed
Roundup Robot added the comment:
New changeset d70995cf44b3 by Christian Heimes in branch '2.7':
Issue #23998: PyImport_ReInitLock() now checks for lock allocation error
https://hg.python.org/cpython/rev/d70995cf44b3
New changeset 7d7bf5c34d7e by Christian Heimes in branch '3.3':
Issue #23998:
Carol Willing added the comment:
Tim, A good point re: the interpreter. I've attached an output from IPython
interpreter.
Georg, Given Tim's additional insight, I'm inclined to reopen the issue and
review the patch as a positive change. Though not a bug, but as an enhancement
for learners to
Joe Jevnik added the comment:
This is a different case from raising an AttributeError inside the __call__;
class C(object):
... def __call__(self):
... raise AttributeError()
...
hasattr(C(), '__call__')
True
class D(object):
... @property
... def __call__(self):
...
Ionel Cristian Mărieș added the comment:
On Sun, Apr 19, 2015 at 10:29 PM, Steven D'Aprano rep...@bugs.python.org
wrote:
This bug report seems to be completely based on a false premise. In the
very first message of this issue, Ionel says:
it return True even if __call__ is actually an
Claudiu Popa added the comment:
Here's the patch.
--
keywords: +patch
stage: needs patch - patch review
Added file: http://bugs.python.org/file39132/issue23917.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23917
R. David Murray added the comment:
Well, previously our answer has been you have to understand unicode. If we
are going to change that, it probably needs a python-dev discussion. But like
I said, providing the *tools* to make it possible to easily do this, just not
as a default, seems like
Ethan Furman added the comment:
Sounds like good incentive to add good docstrings. :)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15582
___
1 - 100 of 136 matches
Mail list logo