[issue12888] html.parser.HTMLParser.unescape works only with the first 128 entities

[Ezio Melotti]

Ezio Melotti  added the comment:

Fixed, thanks for the report and the patch!

--
components: +Library (Lib) -None
resolution:  -> fixed
stage: commit review -> committed/rejected
status: open -> closed
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

[issue12888] html.parser.HTMLParser.unescape works only with the first 128 entities

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 9896fc2a8167 by Ezio Melotti in branch '3.2':
#12888: Fix a bug in HTMLParser.unescape that prevented it to escape more than 
128 entities.  Patch by Peter Otten.
http://hg.python.org/cpython/rev/9896fc2a8167

New changeset 7b6096852665 by Ezio Melotti in branch 'default':
#12888: merge with 3.2.
http://hg.python.org/cpython/rev/7b6096852665

--
nosy: +python-dev

___
Python tracker 

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

[issue12888] html.parser.HTMLParser.unescape works only with the first 128 entities

[Ezio Melotti]

Changes by Ezio Melotti :


--
assignee:  -> ezio.melotti
nosy: +ezio.melotti
stage:  -> commit review
type:  -> behavior

___
Python tracker 

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

[issue12888] html.parser.HTMLParser.unescape works only with the first 128 entities

[Peter Otten]

Peter Otten <__pete...@web.de> added the comment:

The unescape() method uses re.sub(regex, sub, re.ASCII), but the third argument 
is count, not flags. Fix is easy: use

re.sub(regex, sub, flags=re.ASCII).

--
keywords: +patch
nosy: +potten
Added file: http://bugs.python.org/file23092/unescape_bug.patch

___
Python tracker 

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

[issue12888] html.parser.HTMLParser.unescape works only with the first 128 entities

[Yves Dorfsman]

Yves Dorfsman  added the comment:

Added a test case:
http://hg.zioup.org/cpython/rev/4accd3181061

If you set the loop < 128 then the test passes (set at 1000 right now).

--
hgrepos: +65

___
Python tracker 

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

[issue12888] html.parser.HTMLParser.unescape works only with the first 128 entities

[Yves Dorfsman]

New submission from Yves Dorfsman :

html.parser.HTMLParser.unescape works only with the first 128 entities, it 
leaves the other ones as they are.

--
components: None
messages: 143434
nosy: y...@zioup.com
priority: normal
severity: normal
status: open
title: html.parser.HTMLParser.unescape works only with the first 128 entities
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