Gnus development mailing list
 help / color / mirror / Atom feed
From: "Bjørn Mork" <bmork@dod.no>
To: ding@gnus.org
Subject: Re: Make ietf-drums-remove-comments() robust against broken header
Date: Thu, 30 Oct 2008 15:27:13 +0100	[thread overview]
Message-ID: <87prli5fb2.fsf@obelix.mork.no> (raw)
In-Reply-To: <b4mr65ybfki.fsf@jpl.org>

Katsumi Yamaoka <yamaoka@jpl.org> writes:
>>>>>> Bjørn Mork wrote:
>> TSUCHIYA Masatoshi <tsuchiya@namazu.org> writes:
>
>>> I have just met a message including a broken references header, like:
>>>
>>>   References: <200810150047.AA17342@mailhost.example.net>
>>>         <7B77911B-167E-47BF-A826-1C4396FEF000@foo.example.net>
>>>         <7B77911B-167E-47BF-A826-1C4396FEF000@foo.example.net>
>>>         (=?iso-2022-jp?B?IhskQj0pTVUbKEIgGyRCTSdOSRsoQg==?= "'s message of "Fri, 17 Oct 2008 04:32:17 +0900")
>
>> I fail to see how that's broken.  Care to explain?
>
> In the condition where `ietf-drums-syntax-table' is used,
> the parentheses in the last line are unbalanced.  For instance:
>
> (progn
>   (pop-to-buffer "*testing*")
>   (erase-buffer)
>   (insert "(=?iso-2022-jp?B?IhskQj0pTVUbKEIgGyRCTSdOSRsoQg==?= \"'s message of \"Fri, 17 Oct 2008 04:32:17 +0900\")")
>   (require 'ietf-drums)
>   (set-syntax-table ietf-drums-syntax-table)
>   (goto-char (point-min))
>   (forward-sexp 1))

Yes, I can see that the parsing fails, but there's nothing wrong with
the comment, is there?  Any number of "'s are allowed, including odd
numbers.

And won't the suggested patch kill something like this?:

   References: (") <200810150047.AA17342@mailhost.example.net>

which AFAICS is allowed.  



Bjørn
-- 
Ban the war on drugs




  reply	other threads:[~2008-10-30 14:27 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-30  7:11 TSUCHIYA Masatoshi
2008-10-30  8:54 ` Katsumi Yamaoka
2008-10-30  9:07 ` Bjørn Mork
2008-10-30  9:25   ` Katsumi Yamaoka
2008-10-30 14:27     ` Bjørn Mork [this message]
2008-10-30 14:38       ` TSUCHIYA Masatoshi
2008-10-30 23:33         ` Katsumi Yamaoka
2008-10-31  0:35           ` TSUCHIYA Masatoshi
2008-10-31  1:21             ` Katsumi Yamaoka
2008-10-31  3:19               ` Ted Zlatanov
2008-10-31  7:26                 ` Katsumi Yamaoka
2008-10-31  9:06                   ` Katsumi Yamaoka
2008-10-31 17:34                     ` Ted Zlatanov

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=87prli5fb2.fsf@obelix.mork.no \
    --to=bmork@dod.no \
    --cc=ding@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).