Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(15)

Issue 2362041: email5 bytes parsing patch

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 6 months ago by r.david.murray
Modified:
13 years, 5 months ago
Reviewers:
Antoine Pitrou, barry
Base URL:
http://svn.python.org/view/*checkout*/python/branches/py3k/
Visibility:
Public.

Patch Set 1 #

Total comments: 26
Unified diffs Side-by-side diffs Delta from patch set Stats (+515 lines, -78 lines) Patch
M Doc/library/email.rst View 2 chunks +33 lines, -1 line 0 comments Download
M Doc/library/email.generator.rst View 2 chunks +19 lines, -0 lines 0 comments Download
M Doc/library/email.message.rst View 2 chunks +15 lines, -3 lines 0 comments Download
M Doc/library/email.parser.rst View 3 chunks +23 lines, -4 lines 2 comments Download
M Lib/email/__init__.py View 2 chunks +9 lines, -1 line 0 comments Download
M Lib/email/generator.py View 12 chunks +126 lines, -43 lines 8 comments Download
M Lib/email/message.py View 6 chunks +72 lines, -25 lines 11 comments Download
M Lib/email/parser.py View 1 chunk +11 lines, -0 lines 0 comments Download
M Lib/email/test/test_email.py View 3 chunks +207 lines, -1 line 5 comments Download

Messages

Total messages: 4
Antoine Pitrou
This is a quick review. http://codereview.appspot.com/2362041/diff/1/Lib/email/message.py File Lib/email/message.py (right): http://codereview.appspot.com/2362041/diff/1/Lib/email/message.py#newcode29 Lib/email/message.py:29: has_surrogates = re.compile( Do ...
13 years, 6 months ago (2010-10-03 00:28:19 UTC) #1
barry
http://codereview.appspot.com/2362041/diff/1/Doc/library/email.parser.rst File Doc/library/email.parser.rst (right): http://codereview.appspot.com/2362041/diff/1/Doc/library/email.parser.rst#newcode81 Doc/library/email.parser.rst:81: error handler; this data will be recognized as binary ...
13 years, 5 months ago (2010-10-04 17:34:35 UTC) #2
r.david.murray
http://codereview.appspot.com/2362041/diff/1/Doc/library/email.parser.rst File Doc/library/email.parser.rst (right): http://codereview.appspot.com/2362041/diff/1/Doc/library/email.parser.rst#newcode81 Doc/library/email.parser.rst:81: error handler; this data will be recognized as binary ...
13 years, 5 months ago (2010-10-06 01:20:39 UTC) #3
Antoine Pitrou
13 years, 5 months ago (2010-10-06 15:41:23 UTC) #4
http://codereview.appspot.com/2362041/diff/1/Lib/email/message.py
File Lib/email/message.py (right):

http://codereview.appspot.com/2362041/diff/1/Lib/email/message.py#newcode30
Lib/email/message.py:30:
'([^\ud800-\udbff]|\A)[\udc00-\udfff]([^\udc00-\udfff]|\Z)').search
On 2010/10/06 01:20:39, r.david.murray wrote:
> I have very little understanding of what this regex does, I pieced it together
> from things that you and someone else said on IRC.  Please suggest a correct
> regex and I will use it.

I think the following would be enough:

([^\ud800-\udbff]|\A)[\udc80-\udcff]
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b