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

Issue 13244043: mailfs: proper support for utf-8 IMAP searches

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 8 months ago by akumar
Modified:
10 years, 7 months ago
Reviewers:
CC:
rsc_swtch, d, rsc, 0intro, codebot
Visibility:
Public.

Description

UTF-8 searches with the SEARCH command must be conducted in two steps: the first sends the SEARCH command with the length of the UTF-8 encoded string and the second sends the literal search term. The searches need to not be quoted.

Patch Set 1 #

Patch Set 2 : make second step more general #

Patch Set 3 : fix mem leaks: free sx properly #

Unified diffs Side-by-side diffs Delta from patch set Stats (+45 lines, -10 lines) Patch
M src/cmd/upas/nfs/imap.c View 1 2 11 chunks +45 lines, -10 lines 0 comments Download

Messages

Total messages: 5
akumar
10 years, 8 months ago (2013-08-26 21:40:07 UTC) #1
akumar
10 years, 8 months ago (2013-08-26 23:19:38 UTC) #2
akumar
10 years, 8 months ago (2013-08-27 05:36:21 UTC) #3
0intro
I've tested it with Plan 9 imap4d and it seems to work fine.
10 years, 8 months ago (2013-08-27 05:58:17 UTC) #4
rsc
10 years, 7 months ago (2013-09-06 20:09:33 UTC) #5
*** Submitted as
https://code.google.com/p/plan9port/source/detail?r=faec26111cd6 ***

mailfs: support for UTF-8 searches

UTF-8 searches with the SEARCH command must
be conducted in two steps: the first sends
the SEARCH command with the length of the
UTF-8 encoded string and the second sends
the literal search term. The searches need
to not be quoted.

R=rsc, david.ducolombier, rsc, 0intro
CC=plan9port.codebot
https://codereview.appspot.com/13244043

Committer: Russ Cox <rsc@swtch.com>
Sign in to reply to this message.

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