* Small article-unsplit-urls fix
@ 2005-12-01 17:54 Mark Plaksin
2005-12-01 23:16 ` Mark Plaksin
0 siblings, 1 reply; 3+ messages in thread
From: Mark Plaksin @ 2005-12-01 17:54 UTC (permalink / raw)
[-- Attachment #1: Type: text/plain, Size: 175 bytes --]
article-unsplit-urls (W u from the Summary Buffer) turns this:
http://cnn.com
http://ibm.com
into this:
http://cnn.comhttp://ibm.com
Attached is a small patch to fix this.
[-- Attachment #2: gnus-art.el patch --]
[-- Type: text/plain, Size: 504 bytes --]
--- gnus-art.el 2005-11-28 09:37:06.000000000 -0500
+++ gnus-art.el.new 2005-12-01 12:51:02.000000000 -0500
@@ -2494,7 +2494,9 @@
(goto-char (point-min))
(while (re-search-forward
"\\(\\(https?\\|ftp\\)://\\S-+\\) *\n\\(\\S-+\\)" nil t)
- (replace-match "\\1\\3" t)))
+ (if (save-match-data
+ (not (string-match "^\\(https?\\|ftp\\):" (match-string 3))))
+ (replace-match "\\1\\3" t))))
(when (interactive-p)
(gnus-treat-article nil))))
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Small article-unsplit-urls fix
2005-12-01 17:54 Small article-unsplit-urls fix Mark Plaksin
@ 2005-12-01 23:16 ` Mark Plaksin
2006-04-13 6:46 ` Lars Magne Ingebrigtsen
0 siblings, 1 reply; 3+ messages in thread
From: Mark Plaksin @ 2005-12-01 23:16 UTC (permalink / raw)
Mark Plaksin <happy@usg.edu> writes:
> article-unsplit-urls (W u from the Summary Buffer) turns this:
> http://cnn.com
> http://ibm.com
>
> into this:
> http://cnn.comhttp://ibm.com
>
> Attached is a small patch to fix this.
Hmm, now it fails on this:
Hey you should check out http://cnn.com
really!
On second thought it seems there's nothing to fix--people probably only run
it when it's obvious that it will do the right thing.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Small article-unsplit-urls fix
2005-12-01 23:16 ` Mark Plaksin
@ 2006-04-13 6:46 ` Lars Magne Ingebrigtsen
0 siblings, 0 replies; 3+ messages in thread
From: Lars Magne Ingebrigtsen @ 2006-04-13 6:46 UTC (permalink / raw)
Mark Plaksin <happy@usg.edu> writes:
> On second thought it seems there's nothing to fix--people probably only run
> it when it's obvious that it will do the right thing.
Yup. However, I do feel that Gnus should try to do something with
broken URLs automatically.
But I'm not quite sure what. :-)
When being presented with
<http://www.sfgate.com/cgi-bin/article.cgi?file=/c/a/2006/03/25/
MNG6OHU6RR1.DTL&type=printable>
it doesn't take much logic to find out that that's really a single
URL.
Or even this:
http://www.sfgate.com/cgi-bin/article.cgi?file=/c/a/2006/03/25/
MNG6OHU6RR1.DTL&type=printable
However, this isn't:
http://www.sfgate.com/cgi-bin/article.cgi?file=/c/a/2006/03/25/
doesn't work.
In quoted text it gets marginally more difficult:
>> > <http://www.sfgate.com/cgi-bin/article.cgi?file=/c/a/2006/03/25/
>> > MNG6OHU6RR1.DTL&type=printable>
I think some work in this department would be welcome, but I'm not
quite sure where to start.
--
(domestic pets only, the antidote for overdose, milk.)
larsi@gnus.org * Lars Magne Ingebrigtsen
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-04-13 6:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-12-01 17:54 Small article-unsplit-urls fix Mark Plaksin
2005-12-01 23:16 ` Mark Plaksin
2006-04-13 6:46 ` Lars Magne Ingebrigtsen
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).