From: Dave Love <d.love@dl.ac.uk>
Cc: bugs@gnus.org
Subject: Re: one more bug in latest gnus
Date: Fri, 30 May 2003 10:44:04 +0100 [thread overview]
Message-ID: <rzqllwobwgr.fsf@albion.dl.ac.uk> (raw)
In-Reply-To: <rzq8ysqiu6t.fsf@albion.dl.ac.uk> (Dave Love's message of "Wed, 28 May 2003 23:26:50 +0100")
I wrote:
> No better than anyone else, I'm afraid, but it doesn't look right to
> me. `e' is the beginning of the just-decoded word, so the args of
> `mm-decode-coding-region' then appear to be wrong.
Sorry, ignore what I said; I misread the code in a hurry. It's not
very clear without comments.
One thing it shouldn't do is to assume where decode-coding-region
leaves point. As far as I remember, that's not guaranteed (which I
think is a bug).
I suspect that if mm-decode-coding-region must be used, it
could just go before the loop over words, to decode the whole thing
once for all. The chance of QP words containing ASCII that could get
decoded inadvertently is small -- it would have to contain iso2022
escape sequences (or utf-7 if that's defined) -- and it can't happen
for B-encoding. That may be a significant performance improvement,
but I haven't measured it.
next prev parent reply other threads:[~2003-05-30 9:44 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-05-27 13:45 Vladimir Volovich
2003-05-27 21:57 ` Simon Josefsson
2003-05-28 22:26 ` Dave Love
2003-05-30 9:44 ` Dave Love [this message]
[not found] ` <bb7b07$99b$1@quimby.gnus.org>
2003-05-30 17:45 ` Vladimir Volovich
2003-06-02 2:20 ` Jesper Harder
[not found] <bavqsu$46t$1@quimby.gnus.org>
2003-05-27 14:01 ` Vladimir Volovich
2003-05-27 22:02 ` Simon Josefsson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=rzqllwobwgr.fsf@albion.dl.ac.uk \
--to=d.love@dl.ac.uk \
--cc=bugs@gnus.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).