Gnus development mailing list
 help / color / mirror / Atom feed
From: Reiner Steib <reinersteib+gmane@imap.cc>
Subject: gnus-button-alist: Don't treat " as part MID
Date: Tue, 02 Aug 2005 19:57:29 +0200	[thread overview]
Message-ID: <v9hde8dxx2.fsf@marauder.physik.uni-ulm.de> (raw)

Hi,

Gnus treats the trailing quote `"' as part of the MID in case (d) and
(f):

(a) <b4m64uo7ij8.fsf@jpl.org>

(b) news:b4m64uo7ij8.fsf@jpl.org

(c) <news:b4m64uo7ij8.fsf@jpl.org>

(d) "news:b4m64uo7ij8.fsf@jpl.org"

(e) <URL:news:b4m64uo7ij8.fsf@jpl.org>

(f) "URL:news:b4m64uo7ij8.fsf@jpl.org"

AFAICS, `"' can't be part of the domain part, so I suggest apply one
of the following patches.

(1) Minimal change; don't allow `"' ==> @[^>)!;:,\n\t \"]*

--8<---------------cut here---------------start------------->8---
--- gnus-art.el	29 Jul 2005 19:11:46 +0200	6.381.2.46
+++ gnus-art.el	02 Aug 2005 17:23:42 +0200	
@@ -6254,7 +6254,7 @@
 (defcustom gnus-button-alist
   '(("<\\(url:[>\n\t ]*?\\)?\\(nntp\\|news\\):[>\n\t ]*\\([^>\n\t ]*@[^>\n\t ]*\\)>"
      0 (>= gnus-button-message-level 0) gnus-button-handle-news 3)
-    ("\\b\\(nntp\\|news\\):\\([^>\n\t ]*@[^>)!;:,\n\t ]*\\)" 0 t
+    ("\\b\\(nntp\\|news\\):\\([^>\n\t ]*@[^>)!;:,\n\t \"]*\\)" 0 t
      gnus-button-handle-news 2)
     ("\\(\\b<\\(url:[>\n\t ]*\\)?\\(nntp\\|news\\):[>\n\t ]*\\(//\\)?\\([^>\n\t ]*\\)>\\)"
      1 (>= gnus-button-message-level 0) gnus-button-fetch-group 5)
--8<---------------cut here---------------end--------------->8---

(2) The domain part can only contain alphanumeric characters, `.' and
    `-'.  ==> @[a-z0-9.-]+

--8<---------------cut here---------------start------------->8---
--- gnus-art.el	29 Jul 2005 19:11:46 +0200	6.381.2.46
+++ gnus-art.el	02 Aug 2005 17:34:53 +0200	
@@ -6254,7 +6254,7 @@
 (defcustom gnus-button-alist
   '(("<\\(url:[>\n\t ]*?\\)?\\(nntp\\|news\\):[>\n\t ]*\\([^>\n\t ]*@[^>\n\t ]*\\)>"
      0 (>= gnus-button-message-level 0) gnus-button-handle-news 3)
-    ("\\b\\(nntp\\|news\\):\\([^>\n\t ]*@[^>)!;:,\n\t ]*\\)" 0 t
+    ("\\b\\(nntp\\|news\\):\\([^>\n\t ]*@[a-z0-9.-]+\\)" 0 t
      gnus-button-handle-news 2)
     ("\\(\\b<\\(url:[>\n\t ]*\\)?\\(nntp\\|news\\):[>\n\t ]*\\(//\\)?\\([^>\n\t ]*\\)>\\)"
      1 (>= gnus-button-message-level 0) gnus-button-fetch-group 5)
--8<---------------cut here---------------end--------------->8---

I'd prefer (2), but maybe I'm missing something.  Opinions?

Bye, Reiner.

P.S.: The page http://schneegans.de/usenet/mid-schreibweisen/
      summarizes the behavior of different newsreaders (in German;
      "Ja"="Yes" and "Nein"="No").
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/




             reply	other threads:[~2005-08-02 17:57 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-02 17:57 Reiner Steib [this message]
2005-08-04  1:22 ` Katsumi Yamaoka
2005-08-04 12:38   ` Reiner Steib

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=v9hde8dxx2.fsf@marauder.physik.uni-ulm.de \
    --to=reinersteib+gmane@imap.cc \
    --cc=Reiner.Steib@gmx.de \
    /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).