[issue14488] Can't install Python2.7.2

[Éric Araujo]

Éric Araujo  added the comment:

Can you try with 2.7.3?

--
nosy: +eric.araujo

___
Python tracker 

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

[issue14465] xml.etree.ElementTree: add feature to prettify XML output

[Éric Araujo]

Éric Araujo  added the comment:

You may be able to code it entirely in the Python part of the module (adding a 
new parameter to Element.write and tostring).

--
nosy: +eric.araujo

___
Python tracker 

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

[issue14508] gprof2html is broken

[Éric Araujo]

Éric Araujo  added the comment:

Should also use with statements IMO.

--
nosy: +eric.araujo

___
Python tracker 

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

[issue14258] Better explain re.LOCALE and re.UNICODE for \S and \W

[Senthil Kumaran]

Senthil Kumaran  added the comment:

Well, I would like to correct this further and add clarification based on the 
current implementation (_sre.c)

The definition of LOCALE Space is this -

 #define SRE_LOC_IS_SPACE(ch) (!((ch) & ~255) ? isspace((ch)) : 0)

And the definition of NON_SPACE category is a negation of space. That's it.

Now, given that definition, we see for the character values higher than 255, 
the check is not made at all. Is it simple ascii isspace is considered when the 
LOCALE flag is set. And in effect, re.LOCALE flag has not extra effect on 
matching of space or non-white space character.

After realizing this, I propose the following changes attached in the patch as 
a documentation fix.

--
keywords: +patch
resolution: fixed -> 
status: closed -> open
Added file: http://bugs.python.org/file25138/issue14258.diff

___
Python tracker 

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

[issue7057] tkinter doc: more 3.x updates

[Ned Deily]

Ned Deily  added the comment:

The use of Xlib is not limited to Linux; most Unix-y platforms supported by 
Python have Xlib-based Tk versions available.  But it is correct that there are 
Tk implementations that do not use X11 as their window server, for example, the 
native Tk implementations on Windows and Mac OS X.

--
nosy: +ned.deily

___
Python tracker 

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

[issue7057] tkinter doc: more 3.x updates

[Ramchandra Apte]

Ramchandra Apte  added the comment:

"Tkinter uses the Xlib library to draw graphics on the screen" should be 
"Tkinter uses the Xlib library on Linux to draw graphics on the screen"

--
nosy: +ramchandra.apte

___
Python tracker 

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

[issue14502] Document better what happens on releasing an unacquired lock

[Jim Jewett]

Jim Jewett  added the comment:

On Thu, Apr 5, 2012 at 5:38 PM, Antoine Pitrou  wrote:
> Antoine Pitrou  added the comment:

> Not sure what you're talking about. The doc patch is about unacquired
> locks, not locks that someone else (another thread) holds.

Isn't one common reason for not being able to acquire a lock that
someone else was already holding it?

--

___
Python tracker 

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

[issue14492] pdeps.py has_key

[R. David Murray]

R. David Murray  added the comment:

Thanks, Popa.  I made some style changes to the tests, but otherwise used your 
patch.  One small note: your might want to see about setting your editor to 
show whitespace at the ends of lines, or use the 'make patchcheck' command to 
check for whitespace.  Our commit hooks reject files that have lines with 
trailing whitespace, since it tends to clutter up diffs.

--
resolution:  -> fixed
stage: test needed -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue14492] pdeps.py has_key

[Roundup Robot]

Roundup Robot  added the comment:

New changeset e9b8115c5b25 by R David Murray in branch '3.2':
#14492: fix some bugs in Tools/scripts/pdeps.py.
http://hg.python.org/cpython/rev/e9b8115c5b25

New changeset 26a7cc129b3d by R David Murray in branch 'default':
Merge #14492: fix some bugs in Tools/scripts/pdeps.py.
http://hg.python.org/cpython/rev/26a7cc129b3d

--
nosy: +python-dev

___
Python tracker 

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

[issue14243] NamedTemporaryFile unusable under Windows

[R. David Murray]

R. David Murray  added the comment:

See issue 14514 for an alternate proposal to solve this.  I did search before I 
opened that issue, but search is currently somewhat broken and I did not find 
this issue.  I'm not marking it as a dup because my proposal is really a new 
feature.

--
nosy: +r.david.murray

___
Python tracker 

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

[issue14514] Equivalent to tempfile.NamedTemporaryFile that deletes file at context exit

[R. David Murray]

R. David Murray  added the comment:

Grr.  I did a search first, and even used google because I know search is 
currently broken, and I did not find that issue.

--

___
Python tracker 

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

[issue14514] Equivalent to tempfile.NamedTemporaryFile that deletes file at context exit

[Eric V. Smith]

Eric V. Smith  added the comment:

See also issue 14243.

--
nosy: +eric.smith

___
Python tracker 

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

[issue12567] curses implementation of Unicode is wrong in Python 3

[Ned Deily]

Ned Deily  added the comment:

Nicholas, please open a new issue documenting which Python 3.3 you are using, 
from which python.org installer or the ./configure parameters if you built it 
yourself (and whether you supplied a version of GNU readline or used the Apple 
default of BSD libedit) and an example of how to reproduce the error.  Please 
don't add to closed issues.  Note also there is a known open issue with the 
32-bit-only OS X installer for 3.3 where the _curses module does not build 
(Issue14225) with an older version of GNU readline.

--
nosy: +ned.deily

___
Python tracker 

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

[issue14515] tempfile.TemporaryDirectory documented as returning object but returns name

[R. David Murray]

New submission from R. David Murray :

The title pretty much says it all.  I believe the behavior is correct (more 
useful than returning an object that is only useful for obtaining the name) 
even though it is unusual for context managers.  In any case there is plenty of 
code using the existing behavior, so I think this is a doc bug.

--
assignee: docs@python
components: Documentation
messages: 157635
nosy: docs@python, r.david.murray
priority: normal
severity: normal
stage: needs patch
status: open
title: tempfile.TemporaryDirectory documented as returning object but returns 
name
type: behavior
versions: Python 3.2, Python 3.3

___
Python tracker 

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

[issue14507] Segfault with starmap and izip combo

[Raymond Hettinger]

Changes by Raymond Hettinger :


--
assignee:  -> rhettinger

___
Python tracker 

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

[issue14514] Equivalent to tempfile.NamedTemporaryFile that deletes file at context exit

[R. David Murray]

New submission from R. David Murray :

A common pattern (especially in writing tests) is to create a named temporary 
file, operate on it with tools that take the filename, and then delete the 
file.  This pattern would be facilitated by a version of NamedTemporaryFile 
that deleted the named file at the end of the context, but allowed the file to 
continue to exist when closed.  This latter requirement arises because the file 
cannot be opened a second time on Windows, and so needs to be closed before 
other operations that open it can be performed in order for the code to work 
cross platform.

It's tempting just to change delete=True to mean delete on exit (not close) if 
and only if NamedTemporaryFile is being used as a context manager, but there 
might be backward compatibility issues with that.  It might be acceptable as a 
feature change, though, since the likelyhood of someone closing such a tempfile 
inside the context and then depending on it *not* existing between then and the 
end of the context seems low.

If that it deemed unacceptable, this could be implemented as a new flag on 
NamedTemporaryFile, or by changing the delete flag to take additional values 
(True/False == current behavior, but add delete='onclose', delete='onexit', 
delete='no', and document True/False as deprecated.)

--
components: Library (Lib)
messages: 157634
nosy: r.david.murray
priority: normal
severity: normal
status: open
title: Equivalent to tempfile.NamedTemporaryFile that deletes file at context 
exit
type: enhancement
versions: Python 3.3

___
Python tracker 

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

[issue14513] IDLE icon switched and switches on Windows taskbar

[Terry J. Reedy]

Terry J. Reedy  added the comment:

I tried again and 3.2.3rc2 is working. The IDLE icon appears and right clicking 
gives its title as IDLE (Python GUI).

3.3.0a2 is worse than I reported. The reason the icon changes to the pythonw 
icon is because that is what gets pinned, not IDLE. Right clicking the Tk icon 
actually says pythonw. A bare pythonw process is, of course, useless.

--
versions:  -Python 3.2

___
Python tracker 

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

[issue14510] Regular Expression "+" perform wrong repeat

[Ezio Melotti]

Ezio Melotti  added the comment:

>>> re.findall(r'((?:\$.)+)', 'This $b$B is also a variable, but it\'s not 
>>> $B$b')
['$b$B', '$B$b']

--
assignee:  -> ezio.melotti
resolution:  -> invalid
stage:  -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue14508] gprof2html is broken

[R. David Murray]

R. David Murray  added the comment:

I'm getting 'malformed patch at line 30' when I try to apply your patch.

Also, while it's not wrong, I don't think there's much point in catching the 
NameError and doing a fail.  The second Exception clause is definitely wrong, 
though, we don't want other errors to pass silently.   So I think the test 
should just do the call to gprof.main, with a comment mentioning this bug and 
saying that the call used to raise a NameError. (If it does fail for some other 
reason, then the test will need to get a little more complicated so that it 
doesn't.)

--
nosy: +r.david.murray

___
Python tracker 

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

[issue14386] Expose dict_proxy internal type as types.MappingProxyType

[Raymond Hettinger]

Raymond Hettinger  added the comment:

[Victor]
> You mean that I can remove it?

No, it needs to stay there.
Those concrete types aren't exposed to users
but they are used inside to collections.abcs
to register the types so that isinstance()
will work as expected.

Summary:  The collections.abc module is fine as-is.

Go ahead and add dictproxy to the types module.
Expose it there and document it there.

--

___
Python tracker 

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

[issue14417] dict RuntimeError workaround

[STINNER Victor]

STINNER Victor  added the comment:

> The difference is that it's a (presumably interruptible) infinite loop,
> not a stack overflow. There's no crash and therefore no security issue.

Ok, it looks fair. The patch looks good, but as written by Antoine: tests may 
need to be fixed.

Is anyone motivated to "convert" removed crashers to unit tests? Not me. 
Crashers were only written to crash Python. New tests should be written instead.

--

___
Python tracker 

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

[issue14310] Socket duplication for windows

[Santoso Wijaya]

Changes by Santoso Wijaya :


--
nosy: +santa4nt

___
Python tracker 

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

[issue14465] xml.etree.ElementTree: add feature to prettify XML output

[Santoso Wijaya]

Changes by Santoso Wijaya :


--
nosy: +santa4nt

___
Python tracker 

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

[issue12567] curses implementation of Unicode is wrong in Python 3

[STINNER Victor]

STINNER Victor  added the comment:

> AttributeError: '_curses.curses window' object has no attribute 'get_wch'

> is still being raised.

"still"? Did it work before my last changes?

Unicode functions of the (n)curses library are only available if the Python 
curses module is linked to libncursesw.

Is libncursesw available? Is libreadline linked to libncurses or libncursesw? 
If libreadline is linked to libncurses, the Python curses module is also linked 
to libncurses.

--

___
Python tracker 

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

[issue12567] curses implementation of Unicode is wrong in Python 3

[Nicholas Cole]

Nicholas Cole  added the comment:

Testing the Python3.3a2 build on OS X - the exception 


AttributeError: '_curses.curses window' object has no attribute 'get_wch'

is still being raised. I don't have a Linux build I can easily test with. Is 
this a particular problem with the OS X build?

--

___
Python tracker 

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

[issue14502] Document better what happens on releasing an unacquired lock

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> At least put the information inside some disclaimers about "normally";
> even the stdlib has some fake locks that let you release a lock
> someone else holds.

Not sure what you're talking about. The doc patch is about unacquired
locks, not locks that someone else (another thread) holds.

Indeed the standard Lock object (but not the RLock) does allow releasing
from another thread. It's a feature (which makes it serve as a binary
semaphore).

--

___
Python tracker 

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

[issue14502] Document better what happens on releasing an unacquired lock

[Jim Jewett]

Jim Jewett  added the comment:

At least put the information inside some disclaimers about "normally"; even the 
stdlib has some fake locks that let you release a lock someone else holds.  (I 
think I found them in in workarounds for threading not being available, such as 
the dummy_* modules, but still, it is possible.)

--
nosy: +Jim.Jewett

___
Python tracker 

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

[issue14249] unicodeobject.c: aliasing warnings

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

> I just replaced expressions in calls to Py_MAX by variables: Py_MAX is a 
> macro and it may have to compute each expression twice.

gcc computes those values only once. It even caches them for use in
PyUnicode_WRITE. But other compilers may not be so smart.

Instead of Py_MAX(a,b) here you can use a|b. In theory this should be
more efficient, but I couldn't see the difference even with microscope.

However, all this does not matter, soon I will submit complex patch,
which speeds up the utf-16 decoder in 2-5 times.

--

___
Python tracker 

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

[issue14502] Document better what happens on releasing an unacquired lock

[Sandro Tosi]

Changes by Sandro Tosi :


--
resolution:  -> fixed
stage:  -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue14502] Document better what happens on releasing an unacquired lock

[Roundup Robot]

Roundup Robot  added the comment:

New changeset efeca6ff2751 by Sandro Tosi in branch '2.7':
Issue #14502: release() and unlocked lock generates a ThreadError
http://hg.python.org/cpython/rev/efeca6ff2751

New changeset acea9d95a6d8 by Sandro Tosi in branch '3.2':
Issue #14502: release() and unlocked lock generates a ThreadError
http://hg.python.org/cpython/rev/acea9d95a6d8

New changeset c10a0f93544e by Sandro Tosi in branch 'default':
Issue #14502: merge with 3.2
http://hg.python.org/cpython/rev/c10a0f93544e

--
nosy: +python-dev

___
Python tracker 

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

[issue14249] unicodeobject.c: aliasing warnings

[STINNER Victor]

STINNER Victor  added the comment:

Ok, benchmarks have spoken, amen. I applied Serhiy Storchaka's patch (version 
3). I just replaced expressions in calls to Py_MAX by variables: Py_MAX is a 
macro and it may have to compute each expression twice. I didn't check if it's 
more or less efficient. Thanks Serhiy for your contribution! I added your name 
to Misc/ACKS.

We may change Py_MIN/Py_MAX to something more efficient using typeof():

 #define max(a,b) \
   ({ typeof (a) _a = (a); \
   typeof (b) _b = (b); \
 _a > _b ? _a : _b; })

I don't know which C compilers support it, but gcc does, at least.

--

___
Python tracker 

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

[issue9141] Allow objects to decide if they can be collected by GC

[Jim Jewett]

Jim Jewett  added the comment:

I think the second patch is a fairly straightforward enhancement to the 
existing situation.  It still feels a bit like an awkward half-measure, though.

(1)  If a class does have a finalizer, instances should still be able to say 
"Go ahead and collect me, I don't happen to be in the problematic state."  (In 
other words, there should be a return value that means not to bother checking 
for the existence of a tp_del / __del__ method.)

(2)  It should be explicit whether or not this is:
  (2a) just an inquiry ("If you were in a cycle, could I collect you?),
  (2b) a statement plus inquiry ("You are in a cycle.  Can I collect you?"), or
  (2c) a request ("You are in a cycle, and I would like to collect you.  Clean 
up if you can, then tell me whether you are still uncollectable.")

(3)  It may be worth adding an additional slot for safe idempotent finalizers.  
(Earlier suggestions called this __close__, but I suppose __finalize__ might be 
more general.)  Once a garbage cycle is detected, go ahead and call that slot's 
method before giving up and sticking the whole thing in garbage.  If python 
classes could also fill this slot, it would look a lot like (2c).

--
nosy: +Jim.Jewett

___
Python tracker 

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

[issue14249] unicodeobject.c: aliasing warnings

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 489f252b1f8b by Victor Stinner in branch 'default':
Close #14249: Use bit shifts instead of an union, it's more efficient.
http://hg.python.org/cpython/rev/489f252b1f8b

--
resolution:  -> fixed
stage: commit review -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue7057] tkinter doc: more 3.x updates

[Andrew Svetlov]

Changes by Andrew Svetlov :


--
assignee: gpolo -> asvetlov

___
Python tracker 

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

[issue14386] Expose dict_proxy internal type as types.MappingProxyType

[STINNER Victor]

STINNER Victor  added the comment:

> You can ignore those.

You mean that I can remove it?

It's a little bit surprising to find a concrete class in collections.abc. So I 
think that it's better to expose dict_proxy in types than in collections.abc.

--

___
Python tracker 

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

[issue14386] Expose dict_proxy internal type as types.MappingProxyType

[Raymond Hettinger]

Raymond Hettinger  added the comment:

[Victor]
'''Oh, collections.abc contains also the mappingview type exposed with the name 
"dict_proxy": dict_proxy = type(type.__dict__)

It was exposed as _abcoll.dict_proxy in Python 3.2.
It is not documented. Should we keep it for backward compatibility, or can it 
be removed? _abcoll was a private module and I didn't find dict_proxy in Python 
3.2 doc.
'''

You can ignore those.  As you saw, these are undocumented and not listed in 
__all__.  If we ever expose these, it will likely be through the types module 
(or less likely through the collections.abc module which is public in 3.3).  
Probably, they won't get exposed at all because they are awkward to access 
directly and because they have implementation specific details (i.e. other 
implementations have their choice of how to implement various iterators and 
whatnot).

--

___
Python tracker 

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

[issue14417] dict RuntimeError workaround

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> Without a termination condition, how is this different from just
> reverting the original change?

The difference is that it's a (presumably interruptible) infinite loop,
not a stack overflow. There's no crash and therefore no security issue.

--

___
Python tracker 

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

[issue14417] dict RuntimeError workaround

[Jim Jewett]

Jim Jewett  added the comment:

>> A counter can be added to dictobject.c.patch to avoid
>> an infinite loop.

> But why bother? 

Without a termination condition, how is this different from just reverting the 
original change?  Is it just the slightly improvement in efficiency?

--

___
Python tracker 

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

[issue14249] unicodeobject.c: aliasing warnings

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Linux, 64-bit, Intel Core i5 2500:

-> unpatched:

$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = (" " * 
1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 2.99 usec per loop

-> Victor's commit:

$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = (" " * 
1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 2.83 usec per loop

-> utf16_decoder_shift_3.patch:

$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = (" " * 
1000).encode("utf-16be")' 'd(x)'
100 loops, best of 3: 1.88 usec per loop


It seems that the wrong patch was committed.

--
assignee:  -> haypo
resolution: fixed -> 
stage: committed/rejected -> commit review
status: closed -> open

___
Python tracker 

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

[issue14249] unicodeobject.c: aliasing warnings

[Stefan Krah]

Stefan Krah  added the comment:

On 64-bit Linux with gcc-4.4 I get:

Unpatched:

$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = (" " * 
1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 4.1 usec per loop
$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = 
("\u263A" * 1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 5.87 usec per loop

2c514c382a2a:

$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = (" " * 
1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 3.68 usec per loop
$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = 
("\u263A" * 1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 4.72 usec per loop

utf16_decoder_shift_3.patch:

$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = (" " * 
1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 2.23 usec per loop
$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = 
("\u263A" * 1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 3.11 usec per loop

--

___
Python tracker 

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

[issue7839] Popen should raise ValueError if pass a string when shell=False or a list when shell=True

[R. David Murray]

R. David Murray  added the comment:

No, you need to be able to pass bytes to Popen, just like you do to the 
os.exec[xx] functions.  When the OS doesn't fully support unicode, that is 
sometimes the only option.  As for filenames; again, as long as the underlying 
systems use bytes filenames we need to support it.  Currently we encode them 
when received using surrogateescape and decode them back to bytes when used. 

I am not sure what os.exec[xx] does with strings containing non-ascii.  
Presumably it uses some default encoding or other, which seems to be utf-8 on 
my system.  (It doesn't seem to be explicitly documented where those functions 
are discussed in the os module.)

--

___
Python tracker 

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

[issue14249] unicodeobject.c: aliasing warnings

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

What compiler are you using? With gcc 4.4 on 32-bit Linux netbook I get:

 unpatched  union  shift
utf-16le  " "*1 129  126109
utf-16le  "\u263A"*1208  203160
utf-16be  " "*1 153  147114
utf-16be  "\u263A"*1226  227167

The difference is that for shift the compiler stores block in register, and for 
the union the compiler stores block in memory, so that it can get address. May 
be more recent compilers learned to do this more effectively?

Besides, shifts are more pronounced for CPython code, searching shows very few 
uses of union in the source code.

--

___
Python tracker 

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

[issue14513] IDLE icon switched and switches on Windows taskbar

[Terry J. Reedy]

New submission from Terry J. Reedy :

IDLE has an icon that is, appropriately, a white page with text overlaid with 
the Python logo on lower right. (Module docs uses the same icon.) For a long 
time, this has been used everywhere for IDLE -- start menu, desktop, taskbar.
 
In the latest 3.2.3rc2 and 3.3.0a2 releases, the Windows 7 taskbar icon for 
IDLE is the red Tk icon. If IDLE is pinned to the taskbar, the icon changes to 
the command-prompt + logo python-pythonw icon. This seems to be new in these 
releases and is confusing and annoying. Otherwise, the IDLE icon is as it was.

--
components: Installation, Windows
messages: 157611
nosy: georg.brandl, loewis, terry.reedy
priority: normal
severity: normal
status: open
title: IDLE icon switched and switches on Windows taskbar
versions: Python 3.2, Python 3.3

___
Python tracker 

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

[issue14509] Build failures in non-pydebug builds without NDEBUG.

[Antoine Pitrou]

Changes by Antoine Pitrou :


--
nosy: +georg.brandl

___
Python tracker 

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

[issue8515] idle "Run Module" (F5) does not set __file__ variable

[Andrew Svetlov]

Andrew Svetlov  added the comment:

Thanks to all.
Bruce Frederiksen, you are mentioned in Misc/ACK

Tal Einat, if you want to make a patch which will use `execfile` instead of 
current approach — you are welcome. Please file new issue.

--
assignee:  -> asvetlov
resolution:  -> fixed
stage:  -> committed/rejected
status: open -> closed
versions:  -Python 2.7, Python 3.2

___
Python tracker 

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

[issue8515] idle "Run Module" (F5) does not set __file__ variable

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 2c276d0553ff by Andrew Svetlov in branch 'default':
Issue #8515: Set __file__ when run file in IDLE.
http://hg.python.org/cpython/rev/2c276d0553ff

--
nosy: +python-dev

___
Python tracker 

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

[issue7839] Popen should raise ValueError if pass a string when shell=False or a list when shell=True

[Andrew Svetlov]

Andrew Svetlov  added the comment:

What's about test which pass bytes to Popen?
Should it be deprecated and should Popen accept only unicode strings only — I 
mean `str` type?
As I know the trend of py3k to get rid of bytes in filesystem names.

--

___
Python tracker 

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

[issue14505] PyFile_FromString leaks file descriptors in python 2.7

[STINNER Victor]

STINNER Victor  added the comment:

> Victor, what exactly are you talking about? 
> http://hg.python.org/cpython/rev/0f5b64630fda/ *does* change 
> PyFile_FromString.

Oh. I don't know (remember) why I did this change!? I suppose that I
changed it to test something and then forget to remove the temporary
hack :-/

--

___
Python tracker 

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

[issue14512] Pydocs module docs server not working on Windows.

[Terry J. Reedy]

New submission from Terry J. Reedy :

3.2.3rc2 and 3.3.0a2, freshly downloaded and installed, Win7, 64 bit
Start Menue/Pythonxx/Module Docs
Brings up tkinter pydoc box.
[open browser] brings up *empty* localhost:7464 window.
Search 'tkinter' and double-click tkinter entry or select and [go to selected] 
opens empty localhost:7464/tkinter.html window.

This worked sometime last year on earlier 3.2, maybe on xp box.

I thought there might be an issue already, but I searched for 'pydoc' and 
'modole docs' and found nothing.

--
assignee: docs@python
components: Documentation, Installation, Library (Lib)
messages: 157606
nosy: docs@python, georg.brandl, loewis, terry.reedy
priority: normal
severity: normal
stage: needs patch
status: open
title: Pydocs module docs server not working on Windows.
type: behavior
versions: Python 3.2, Python 3.3

___
Python tracker 

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

[issue14417] dict RuntimeError workaround

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> Does this mean I should just check it in? But I asked, and never got
> an answer, whether the original stress test had been converted into a
> unittest. I'd like that to happen before I check this in. Also there
> are probably docs I've missed. Somebody please help!

I think your approach is fine. As far as I can tell, the original
crasher hasn't been converted into an unit test, since Victor's test in
934aaf2191d0 doesn't seem prone to triggering an infinite loop.

As far as doc changes go, you should probably remove the what's new
entry from a428f85de29c, and the doc addition from 0255bafbccf2.

Besides, does the test suite pass with your patch? It looks like
Victor's test, which checks that RuntimeError is raised, should now
fail.

--

___
Python tracker 

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

[issue14417] dict RuntimeError workaround

[Guido van Rossum]

Guido van Rossum  added the comment:

> A counter can be added to dictobject.c.patch to avoid an infinite loop.

But why bother? There was no counter originally -- it would continue
until it ran out of stack. Since this can only be triggered if there
is Python code in the __eq__, that means that it is still
interruptable with ^C.

Does this mean I should just check it in? But I asked, and never got
an answer, whether the original stress test had been converted into a
unittest. I'd like that to happen before I check this in. Also there
are probably docs I've missed. Somebody please help!

--

___
Python tracker 

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

[issue14478] Decimal hashing very slow, could be cached

[James Hutchison]

James Hutchison  added the comment:

I presume you mean in 3.2? Have you looked at the source code for that 
decorator? It's fundamentally a try/except but with a lot more unnecessary 
bloat than is needed for caching a single int result from a function with no 
arguments. Its actually a lot slower.

If this is likely going to see use in 3.3 then it would probably just be a long 
int since 3.3 uses C. 0 would indicate uncalculated.

Hash function would have to be set up to never return 0.

Also every function would need to be tested to make sure there isn't any 
"in-place" modification of the Decimal object that could alter the hash value.

I like how the cached hash in 3.3 is faster than int for hashing. Shouldn't an 
int just return itself? Why would it be slower?

--

___
Python tracker 

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

[issue11060] distutils2 sdist does not complain about version that is not PEP 386 compliant

[Éric Araujo]

Éric Araujo  added the comment:

Thanks for the tests!  I should have some time this week-end to review them and 
explain clearly what this bug is about and where tests should go.

--

___
Python tracker 

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

[issue14506] HTMLParser can't handle erronous end tags with additional info in them

[Ezio Melotti]

Ezio Melotti  added the comment:

This is already fixed, but only in non-strict mode (and 3.2.3 iirc).
You should always use HTMLParser(strict=False).  The non-strict mode will 
probably become the default and strict=True will be deprecated.
Thanks anyway for the report, and please report any failure that you might find 
with strict=False.

--
resolution:  -> invalid
stage:  -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue14510] Regular Expression "+" perform wrong repeat

[Matthew Barnett]

Matthew Barnett  added the comment:

If a capture group is repeated, as in r'(\$.)+', only its last match is 
returned.

--

___
Python tracker 

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

[issue14511] _static/opensearch.xml for Python 3.2 docs directs searches to 3.3 docs

[Zachary Ware]

New submission from Zachary Ware :

Adding the search plugin for the 3.2 docs to Firefox and then searching from it 
returns results from the 3.3 dev docs, despite everything saying it should be 
for searching "Python v3.2.2 documentation".

If my understanding of how it all works is correct, the attached patch should 
fix the issue.  It simply removes 'dev/' from the html_use_opensearch 
assignment in Doc\conf.py in the 3.2 branch.

Thanks,

Zach

--
assignee: docs@python
components: Documentation
files: 3.2 conf.py opensearch.diff
keywords: patch
messages: 157599
nosy: docs@python, eric.araujo, ezio.melotti, georg.brandl, zach.ware
priority: normal
severity: normal
status: open
title: _static/opensearch.xml for Python 3.2 docs directs searches to 3.3 docs
type: behavior
versions: Python 3.2
Added file: http://bugs.python.org/file25137/3.2 conf.py opensearch.diff

___
Python tracker 

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

[issue14452] SysLogHandler sends invalid messages when using unicode

[marko kreen]

marko kreen  added the comment:

Note additional brokenness in BOM addition:
* It does add BOM even when msg is ascii, just because it was in unicode() 
string.
* It does not add BOM to UTF8 string if it is already encoded to str().

So highly doubt anybody actually relies on it.  And keeping such addition just 
increases chance that anybody starts to rely on it, so it would be good to just 
drop it.

For 3.3, I think it would be best to move syslog packet formatting out from 
emit(), because emit() already contains noticeable amount of unrelated code, so 
full override it is annoying.

I suggested changing BOM addition so that it only adds it to %(message)s, but 
such change could cause backwards incompatibility. So it does not look like 
good idea.

But dropping BOM even in old Python *does* look like good idea, as anybody 
using SysLoghandler needs to deal with BOM-less UTF8 anyway,
so dropping it will not create backwards incompatibility.

--

___
Python tracker 

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

[issue14452] SysLogHandler sends invalid messages when using unicode

[Vinay Sajip]

Vinay Sajip  added the comment:

Ok, I see what the problem is. I could go for option 1 - leave the BOM out, 
encode the string as UTF-8 but send it as just a bunch of bytes, i.e. the 
MSG-ANY variant of the spec. However, this could break any existing code that 
doesn't use structured data before the message, as you are doing, and relies on 
the MSG-UTF8 variant. So while agreeing with you that the situation isn't 
ideal, I don't see how I can change things while preserving backward 
compatibility, other than:

Introduce a class-level _insert_BOM attribute, defaulting to True but which can 
be set to False on a per-instance level. The BOM would only be inserted if this 
were True, so the current behaviour is preserved, but you could set the 
attribute to False and leave the BOM out. However, isn't ideal either, as it 
requires you to be sensitive to the exact version of Python in use - not easy 
if you're developing a library rather than an application.

I will consider a different approach in 3.3, e.g. provide one or more 
overridable methods to construct the message sent across the socket.

Thoughts? If these methods won't suffice, you can always resort to subclassing 
the handler and overriding the emit method (not that that's ideal, either).

--
resolution: invalid -> 

___
Python tracker 

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

[issue14503] docs:Code not highlighted

[R. David Murray]

R. David Murray  added the comment:

Huh.  Well, in another issue Georg said it was now possible to upgrade the doc 
build toolchain to Python3.

--

___
Python tracker 

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

[issue14503] docs:Code not highlighted

[Amaury Forgeot d'Arc]

Amaury Forgeot d'Arc  added the comment:

Not exactly. Sphinx first tries to see if the block is a Python script, and to 
do so it uses the Python compiler:
https://bitbucket.org/birkenfeld/sphinx/src/164f59b2d946/sphinx/highlighting.py#cl-117
In case of SyntaxError, it treats the whole block as plain text and does not 
try to use pygmentq at all.

--

___
Python tracker 

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

[issue1047540] Turtle.py hangs Idle

[Andrew Svetlov]

Andrew Svetlov  added the comment:

Closing as not reproducible for 3.3. 
The bug is too minor to worry about 2.7 and 3.2 even if it present (all works 
for me by the way).
Please reopen if you will get described issue.

--
assignee:  -> asvetlov
nosy: +asvetlov
resolution:  -> out of date
stage: test needed -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue14503] docs:Code not highlighted

[R. David Murray]

R. David Murray  added the comment:

Ah, you mean they are not *syntax* highlighted.  Now I understand.  Sorry for 
missing that.

My understanding is that Sphinx does not use Python directly to parse the code 
and highlight it, it uses pygments, which uses regexes.  So this basically 
amounts to a feature request for pygments.

--

___
Python tracker 

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

[issue1542677] IDLE shell gives different len() of unicode strings compared to Python shell

[Andrew Svetlov]

Andrew Svetlov  added the comment:

Closing as won't fix.

--
assignee: kbk -> asvetlov
nosy: +asvetlov
resolution:  -> wont fix
stage: test needed -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue14496] Wrong name in idlelib/tabbedpages.py

[Andrew Svetlov]

Andrew Svetlov  added the comment:

Fixed. Thanks.

--
resolution:  -> fixed
stage:  -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue14496] Wrong name in idlelib/tabbedpages.py

[Roundup Robot]

Roundup Robot  added the comment:

New changeset f2dfe0ca6c21 by Andrew Svetlov in branch '3.2':
Issue #14496: Fix wrong name in idlelib/tabbedpages.py.
http://hg.python.org/cpython/rev/f2dfe0ca6c21

--
nosy: +python-dev

___
Python tracker 

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

[issue14503] docs:Code not highlighted

[Amaury Forgeot d'Arc]

Amaury Forgeot d'Arc  added the comment:

> they are highlighted
No. Keywords normally appear in bold font, and with another color. This is not 
the case in these blocks.

Probably because the language detection does not work with the new "yield from" 
construct.  Sphinx should always use the latest Python...

--
nosy: +amaury.forgeotdarc

___
Python tracker 

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

[issue14510] Regular Expression "+" perform wrong repeat

[YunJian]

New submission from YunJian :

Regular expression perform wrong result, I use regular expression r'(\$.)+' try 
to match "$B$b" or something like that, but the script only give back "$b", 
r'(\$.){1,}', r'(\$.){2}' performed the same, you can take the file I attached 
for reference and run it, I try to solve it myself but failed, now I must use 
other ways to realize my aim, so appreciate for you help, thank you!

--
components: Regular Expressions
files: CheckInconsistency.py
messages: 157588
nosy: ezio.melotti, mrabarnett, tld5yj
priority: normal
severity: normal
status: open
title: Regular Expression "+" perform wrong repeat
type: behavior
versions: Python 3.2
Added file: http://bugs.python.org/file25136/CheckInconsistency.py

___
Python tracker 

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

[issue14503] docs:Code not highlighted

[R. David Murray]

R. David Murray  added the comment:

Berker: you can reproduce the bug, or the fact that they are highlighted?  The 
png looks like they are highlighted, so I assume the latter.

--

___
Python tracker 

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

[issue14509] Build failures in non-pydebug builds without NDEBUG.

[Thomas Wouters]

New submission from Thomas Wouters :

The hash randomization change conflates 'pydebug' builds (which define the 
'Py_DEBUG' preprocessor symbol) and asserts (which use 'NDEBUG' instead.) While 
Py_DEBUG automatically unsets NDEBUG, the inverse is not true (and should not 
be true.)

In random.c (and Include/object.h), _Py_HashSecret_Initialized is defined as 
static or global depending on the value of Py_DEBUG. But in (at least) 
Objects/stringobject.c and Objects/unicodeobject.c, unguarded asserts are used 
that check the value of _Py_HashSecret_Initialized. This causes builds that 
define neither NDEBUG nor Py_DEBUG to fail. Either the guard used around the 
declaration and definition should be '#ifndef NDEBUG' instead of '#ifdef 
Py_DEBUG', or there should be a Py_DEBUG guard around the two assert statements.

(I would submit a patch but my development environment died during a nasty 
power outage yesterday.)

--
components: Interpreter Core
messages: 157586
nosy: benjamin.peterson, twouters
priority: release blocker
severity: normal
status: open
title: Build failures in non-pydebug builds without NDEBUG.
type: compile error
versions: Python 2.7

___
Python tracker 

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

[issue14506] HTMLParser can't handle erronous end tags with additional info in them

[R. David Murray]

R. David Murray  added the comment:

I just tested your script on 3.2.3a2+, and it raises an error.  Ezio made the 
other parsing changes, I'll leave it to him to evaluate what if anything should 
be done here.

--
versions: +Python 3.3

___
Python tracker 

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

[issue14503] docs:Code not highlighted

[Berker Peksag]

Berker Peksag  added the comment:

I can reproduce. See screenshot.png.

--
nosy: +berker.peksag
Added file: http://bugs.python.org/file25135/screenshot.png

___
Python tracker 

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

[issue14506] HTMLParser can't handle erronous end tags with additional info in them

[Olaf Tomalka]

Olaf Tomalka  added the comment:

Python 3.2.2, which is latest on arch linux

--

___
Python tracker 

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

[issue14506] HTMLParser can't handle erronous end tags with additional info in them

[R. David Murray]

R. David Murray  added the comment:

Which version of python did you test with?  There have been several 
improvements html parsing recently.

--
nosy: +ezio.melotti, r.david.murray

___
Python tracker 

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

[issue11060] distutils2 sdist does not complain about version that is not PEP 386 compliant

[Rik Poggi]

Rik Poggi  added the comment:

Strictly related or not to this bug, a bit more test coverage shouldn't hurt.

So while waiting for a reply I started writing a couple of tests for 
pypi/dist.py, hope they look good.

--
Added file: http://bugs.python.org/file25134/test_pypi_dist.diff

___
Python tracker 

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

[issue14508] gprof2html is broken

[Popa Claudiu]

New submission from Popa Claudiu :

Tools/gprof2html.py uses "file" to open a file. Also, the opened file passed to 
add_escapes was never closed. There's a test included in the patch.

--
components: Demos and Tools
files: gprof.patch
keywords: patch
messages: 157580
nosy: Popa.Claudiu
priority: normal
severity: normal
status: open
title: gprof2html is broken
type: behavior
versions: Python 3.2, Python 3.3
Added file: http://bugs.python.org/file25133/gprof.patch

___
Python tracker 

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

[issue14504] Suggestion to improve argparse's help messages for "store_const"

[R. David Murray]

Changes by R. David Murray :


--
nosy: +bethard
versions: +Python 3.3 -Python 2.7

___
Python tracker 

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

[issue14503] docs:Code not highlighted

[R. David Murray]

R. David Murray  added the comment:

I'm not seeing any unhighlighted examples in that section.

--
nosy: +r.david.murray

___
Python tracker 

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

[issue14507] Segfault with starmap and izip combo

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Apparently it's a stack overflow between zip_next and starmap_next.

--
nosy: +pitrou, rhettinger
versions: +Python 3.3

___
Python tracker 

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

[issue14501] Error initialising BaseManager class with 'authkey' argument of string type.

[R. David Murray]

R. David Murray  added the comment:

Thanks for the report.

I'm not familiar with multiprocessing, so I'll have to leave it to someone else 
to judge the fix.

We use 'crash' to indicate a segfault in the Python interpreter, so I'm 
changing the type to 'behavior' (that is, a normal bug).

--
nosy: +asksol, r.david.murray
type: crash -> behavior
versions: +Python 3.3

___
Python tracker 

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

[issue14507] Segfault with starmap and izip combo

[Per Myren]

New submission from Per Myren :

The following code crashes with a segfault on Python 2.7.2:

from operator import add
from itertools import izip, starmap

a = b = [1]
for i in xrange(10):
a = starmap(add, izip(a, b))

list(a)


It also crashes with Python 3.2.2:

from operator import add
from itertools import starmap

a = b = [1]
for i in range(10):
a = starmap(add, zip(a, b))

list(a)

--
components: Library (Lib)
messages: 157576
nosy: progrper
priority: normal
severity: normal
status: open
title: Segfault with starmap and izip combo
type: crash
versions: Python 2.7, Python 3.2

___
Python tracker 

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

[issue14506] HTMLParser can't handle erronous end tags with additional info in them

[Olaf Tomalka]

Changes by Olaf Tomalka :


--
title: HTMLParser can't handle erronous end tags with additional tags in it -> 
HTMLParser can't handle erronous end tags with additional info in them

___
Python tracker 

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

[issue14505] PyFile_FromString leaks file descriptors in python 2.7

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Matthias, this should be fixed now.

--
resolution:  -> fixed
stage:  -> committed/rejected
status: open -> closed
type:  -> resource usage

___
Python tracker 

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

[issue14505] PyFile_FromString leaks file descriptors in python 2.7

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 8258e5fa4a19 by Antoine Pitrou in branch '2.7':
Issue #14505: Fix file descriptor leak when deallocating file objects created 
with PyFile_FromString().
http://hg.python.org/cpython/rev/8258e5fa4a19

--
nosy: +python-dev

___
Python tracker 

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

[issue14385] Support other types than dict for __builtins__

[STINNER Victor]

STINNER Victor  added the comment:

New version:
 - if __build_class__ is missing, raise a NameError instead of surprising 
ImportError
 - add tests
 - if PyObject_GetItem on __builtins__ or globals fail, only raise NameError if 
the exception is a KeyError

Before my patch, a new dict was created for builtins if __builtins__ exists in 
global but is not a dict. With my patch, the __builtins__ is kept and the type 
is checked at runtime. If __builtins__ is not a mapping, an exception is raised 
on lookup in ceval.c.

We may check __builtins__ type in PyFrame_New() using:

PyDict_Check(builtins) || (PyMapping_Check(mapping) && !PyList_Check(mapping) 
&& !PyTuple_Check(mapping))

(PyDict_Check(builtins) is checked first for performance)

--
Added file: http://bugs.python.org/file25132/builtins-2.patch

___
Python tracker 

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

[issue9141] Allow objects to decide if they can be collected by GC

[Andrew Svetlov]

Changes by Andrew Svetlov :


--
nosy: +asvetlov

___
Python tracker 

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

[issue14386] Expose dict_proxy internal type as types.MappingProxyType

[STINNER Victor]

Changes by STINNER Victor :


--
title: Expose dict_proxy internal type as types.MappingViewType -> Expose 
dict_proxy internal type as types.MappingProxyType

___
Python tracker 

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

[issue14452] SysLogHandler sends invalid messages when using unicode

[marko kreen]

marko kreen  added the comment:

The 'msg' in SysLogHandler does not correspond to MSG in RFC.
Nor to %(message)s in log record.

RFC:

SYSLOG-MSG  = HEADER SP STRUCTURED-DATA [SP MSG]
HEADER  = PRI VERSION SP TIMESTAMP SP HOSTNAME
  SP APP-NAME SP PROCID SP MSGID
PRI = "<" PRIVAL ">"
[...]
MSG = MSG-ANY / MSG-UTF8
MSG-ANY = *OCTET ; not starting with BOM
MSG-UTF8= BOM UTF-8-STRING
BOM = %xEF.BB.BF


logging:

The SysLogHandler does not provide any fields after PRI.
Which is OK, those can be added via format string.

We are using following formatter to get RFC-like structure
to message:

  [formatter_syslog]
  format=%(hostname)s %(service_name)s %(job_name)s %(message)s

That's not fully compliant, but that's beside the point.
The problem is that SysLogHandler puts BOM before hostname
not before %(message)s.  In Python 2.6 it put it even before PRI,
which was even more annoying.

I see 2 solutions to this:

1) Stop putting BOM at all, its not necessary.
2) Put the BOM into %(message)s somehow, thus working with whatever
format is given.

--
status: pending -> open

___
Python tracker 

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

[issue9141] Allow objects to decide if they can be collected by GC

[Kristján Valur Jónsson]

Kristján Valur Jónsson  added the comment:

Here is a completely new patch.  This approach uses the already existing 
tp_is_gc enquiry slot to signal garbage collection.
The patch modifies the generator object to use this new mechanism.
The patch keeps the old PyGen_NeedsFinalizing() API, but this can now go away, 
unless people think it might be used in extension modules 

(why do we always expose all those internal apis from the dll? I wonder.)

--
Added file: http://bugs.python.org/file25131/ob_is_gc.patch

___
Python tracker 

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

[issue14506] HTMLParser can't handle erronous end tags with additional tags in it

[Olaf Tomalka]

New submission from Olaf Tomalka :

While this is wrongly formated html, I've spotted such an example on real 
website on the web, and all browsers handle the bad tag gracefully, while the 
python html parser throws an exception with "bad end tag", I think additional 
info in end tag should be ignored, no exception thrown and rest of the page 
parsed.
I'm including minimal example.

--
components: Library (Lib)
files: minimal.py
messages: 157570
nosy: ritave
priority: normal
severity: normal
status: open
title: HTMLParser can't handle erronous end tags with additional tags in it
type: behavior
versions: Python 3.2
Added file: http://bugs.python.org/file25130/minimal.py

___
Python tracker 

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

[issue1602] windows console doesn't print or input Unicode

[STINNER Victor]

STINNER Victor  added the comment:

The issue #14227 has been marked as a duplicate of this issue. Copy of 
msg155149:

This is on Windows 7 SP1.  Run 'chcp 65001' then Python from a console.  Note 
the extra characters when non-ASCII characters are in the string.  At a guess 
it appears to be using the UTF-8 byte length of the internal representation 
instead of the character count.

Python 3.3.0a1 (default, Mar  4 2012, 17:27:59) [MSC v.1500 32 bit (Intel)] on 
win32
Type "help", "copyright", "credits" or "license" for more information.
>>> print('hello')
hello
>>> print('p\u012bny\u012bn')
pīnyīn
n
>>> print('\u012b'*10)
īī
�
�ī

--

___
Python tracker 

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

[issue14227] console w/ cp65001 displays extra characters for non-ascii strings.

[STINNER Victor]

STINNER Victor  added the comment:

I'm quite sure that this issue is a duplicate of #1602.

--
resolution:  -> duplicate
status: open -> closed

___
Python tracker 

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

[issue14249] unicodeobject.c: aliasing warnings

[STINNER Victor]

STINNER Victor  added the comment:

Result of the benchmark before/after my commit. I prefer an unit over manually 
manipulate long as short or bytes, because I think that the compiler knows 
better how to optimize operations on integers.

unpatched:

$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = (" " * 
1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 4.64 usec per loop
$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = 
("\u263A" * 1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 5.87 usec per loop

patched:

$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = (" " * 
1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 3.53 usec per loop
$ ./python -m timeit -s 'import codecs; d = codecs.utf_16_be_decode; x = 
("\u263A" * 1000).encode("utf-16be")' 'd(x)'
10 loops, best of 3: 4.85 usec per loop

--

___
Python tracker 

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

[issue14249] unicodeobject.c: aliasing warnings

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 2c514c382a2a by Victor Stinner in branch 'default':
Close #14249: Use an union instead of a long to short pointer to avoid aliasing
http://hg.python.org/cpython/rev/2c514c382a2a

--
nosy: +python-dev
resolution:  -> fixed
stage:  -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue14222] Use time.steady() to implement timeout

[STINNER Victor]

STINNER Victor  added the comment:

> You can imagine a similar patch for Queue timeout.

Oops: You can imagine a similar *test* for Queue timeout.

--

___
Python tracker 

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

[issue14222] Use time.steady() to implement timeout

[STINNER Victor]

STINNER Victor  added the comment:

> Why do you think monotonic time is needed for the Queue module?

The duration of a timeout of N seconds should be N seconds even if the system 
clock is updated (e.g. a daylight saving time (DST) change).

This feature is checked by an unit test in the patch attached to the isuse 
#14428 (implementation of the PEP 418):

+def test_monotonic_settime(self):
+t1 = time.monotonic()
+realtime = time.clock_gettime(time.CLOCK_REALTIME)
+# jump backward with an offset of 1 hour
+time.clock_settime(time.CLOCK_REALTIME, realtime - 3600)
+t2 = time.monotonic()
+time.clock_settime(time.CLOCK_REALTIME, realtime)
+# monotonic must not be affected by system clock updates
+self.assertGreaterEqual(t2, t1)

You can imagine a similar patch for Queue timeout.

--

___
Python tracker 

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

[issue14505] PyFile_FromString leaks file descriptors in python 2.7

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Victor, what exactly are you talking about? 
http://hg.python.org/cpython/rev/0f5b64630fda/ *does* change PyFile_FromString.
And Matthias mentioned that the problem didn't occur with 2.6.

--

___
Python tracker 

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

[issue14417] dict RuntimeError workaround

[STINNER Victor]

STINNER Victor  added the comment:

A counter can be added to dictobject.c.patch to avoid an infinite loop.

--

___
Python tracker 

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

[issue14505] PyFile_FromString leaks file descriptors in python 2.7

[STINNER Victor]

STINNER Victor  added the comment:

> Victor, that sounds like a strange behaviour to me.

We may change PyFile_FromString() to call fclose() when the file is
closed explicitly (call its close() method), but it may break backward
compatibility. I prefer to document the behaviour instead.

> PyFile_FromString is a public API and maybe it shouldn't
> have changed between 2.6 and 2.7.

PyFile_FromString() didn't change in Python 2.7.

I changed PyFile_FromFile() in Python 2.7 to fix the issue #7732.

changeset:   72456:0f5b64630fda
branch:  2.7
parent:  72450:c02e790c4535
user:Victor Stinner 
date:Fri Sep 23 19:37:03 2011 +0200
files:   Doc/c-api/file.rst Lib/test/test_import.py Misc/NEWS
Objects/fileobject.c Python/import.c
description:
Issue #7732: Fix a crash on importing a module if a directory has the same name
than a Python module (e.g. "__init__.py"): don't close the file twice.

PyFile_FromFile() does also close the file if PyString_FromString() failed. It
did already close the file on fill_file_fields() error (e.g. if the file is a
directory).

--

___
Python tracker 

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

[issue14505] PyFile_FromString leaks file descriptors in python 2.7

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Victor, that sounds like a strange behaviour to me. PyFile_FromString is a 
public API and maybe it shouldn't have changed between 2.6 and 2.7.

--

___
Python tracker 

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

[issue14505] PyFile_FromString leaks file descriptors in python 2.7

[STINNER Victor]

STINNER Victor  added the comment:

> File descriptors opened by PyFile_FromString don't get closed
> when the reference count is decreased.

Correct, PyFile_FromString() doesn't close the file descriptor, even if you 
call its close() method. You have to call manually fclose(file->f_fp).

Or you can use PyFile_FromFile:

fp = fopen(name, mode);
if (fp == NULL) ...
obj = PyFile_FromFile(fp, name, mode, fclose);
if (obj == NULL) {
   /* no need to call fclose(fp) here, it's done by PyFile_FromFile() */
   ...
}
...
Py_DECREF(obj);

Would you like to write a patch for the documentation?

--
nosy: +pitrou

___
Python tracker 

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