[issue2927] expose html.parser.unescape

Ezio Melotti added the comment: Fixed, thanks for the reviews! -- resolution: -> fixed stage: commit review -> committed/rejected status: open -> closed ___ Python tracker ___ __

[issue2927] expose html.parser.unescape

Roundup Robot added the comment: New changeset 7b9235852b3b by Ezio Melotti in branch 'default': #2927: Added the unescape() function to the html module. http://hg.python.org/cpython/rev/7b9235852b3b -- nosy: +python-dev ___ Python tracker

[issue2927] expose html.parser.unescape

Serhiy Storchaka added the comment: LGTM. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue2927] expose html.parser.unescape

Ezio Melotti added the comment: Here is the last iteration with a few minor tweaks and a couple more tests. -- stage: patch review -> commit review Added file: http://bugs.python.org/file32703/issue2927-3.diff ___ Python tracker

[issue2927] expose html.parser.unescape

Ezio Melotti added the comment: Here's an updated patch that addresses comments on rietveld and adds a few more tests and docs. I should also update the what's new, but I have other upcoming changes in the html package so I'll probably do it at the end. Regarding your concern: * if people are

[issue2927] expose html.parser.unescape

Serhiy Storchaka added the comment: I added comments on Rietveld. Yet one thing. For now the html module is very simple and has no dependencies. The patch adds an import of re and html.escapes and relative heavy re.compile operations. Due to the fact that the html module is implicitly imported

[issue2927] expose html.parser.unescape

Changes by Ezio Melotti : -- assignee: -> ezio.melotti versions: +Python 3.4 -Python 3.2 Added file: http://bugs.python.org/file32682/issue2927.diff ___ Python tracker ___ __

[issue2927] expose html.parser.unescape

Changes by Martin Panter : -- nosy: +vadmium ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.

[issue2927] expose html.parser.unescape

Tom Pinckney added the comment: New patch attached, tested against Python 3.2. This is my first Python patch so apologies if I've done something wrong here. Feedback appreciated! Changes: * fit everything to 80 cols * just made changes to the HTMLParser.unescape function instead of providing

[issue2927] expose html.parser.unescape

Tom Pinckney added the comment: I don't think Django includes an HTML unescape. I'm not familiar with other frameworks. So I'd still find this useful to include in the stdlib. -- ___ Python tracker ___

[issue2927] expose html.parser.unescape

STINNER Victor added the comment: I'm not sure that using an hardcoded mapping CP1252 => unicode is a good idea. -- ___ Python tracker ___ ___

[issue2927] expose html.parser.unescape

Changes by Éric Araujo : -- nosy: +ezio.melotti, haypo status: pending -> open versions: +Python 3.2 -Python 2.7 ___ Python tracker ___ ___

[issue2927] expose html.parser.unescape

Mark Lawrence added the comment: msg110657 recommends close -> wontfix. Does anybody want this kept open or can it be closed? -- status: open -> pending ___ Python tracker ___

[issue2927] expose html.parser.unescape

Reid Kleckner added the comment: It's using the old Python 2 unicode string literal syntax. It also doesn't keep to 80 cols. I'd also rather continue using a lazily initialized dict instead of catching a KeyError for '. I also feel that with the changes to Unicode in py3k, the cp1252 stuff w

[issue2927] expose html.parser.unescape

Mark Lawrence added the comment: Trying to run the test and I get:- c:\py3k\Lib>..\PCbuild\python_d.exe test\test_htmlparser.py File "test\test_htmlparser.py", line 326 escaped = u"There’s the Côte" ^ SyntaxError: invalid syntax -

[issue2927] expose html.parser.unescape

Changes by Brian Curtin : -- keywords: +needs review priority: -> normal stage: -> patch review versions: +Python 2.7 -Python 2.6 ___ Python tracker ___

[issue2927] expose html.parser.unescape

Brett Cannon <[EMAIL PROTECTED]> added the comment: The plan is to add html.escape(). Adding html.unescape() wouldn't hurt. -- nosy: +brett.cannon __ Tracker <[EMAIL PROTECTED]> __ ___

[issue2927] expose html.parser.unescape

New submission from Tom Pinckney <[EMAIL PROTECTED]>: There is currently a private method inside of html.parser.HTMLParser to unescape HTML &...; style escapes. This would be useful to expose for other users who want to unescape a piece of HTML. Additionally, many websites don't use proper uni