* [patch]
@ 1999-01-04 19:40 Colin Rafferty
1999-01-11 1:04 ` [patch] Lars Magne Ingebrigtsen
0 siblings, 1 reply; 5+ messages in thread
From: Colin Rafferty @ 1999-01-04 19:40 UTC (permalink / raw)
[-- Attachment #1: Type: text/plain, Size: 50 bytes --]
Deleting a long line was being done incorrectly.
[-- Attachment #2: patch to nnmail --]
[-- Type: text/plain, Size: 499 bytes --]
1999-01-04 Colin Rafferty <colin@xemacs.org>
* nnmail.el (nnmail-article-group): Do pathologically long line
splitting correctly.
--- nnmail.el~ Sun Jan 3 08:36:06 1999
+++ nnmail.el Mon Jan 4 14:37:40 1999
@@ -1069,7 +1069,7 @@
(while (not (eobp))
(end-of-line)
(if (> (current-column) nnmail-split-header-length-limit)
- (delete-region (point) (progn (end-of-line) (point)))
+ (gnus-delete-line)
(forward-line 1)))
;; Allow washing.
(goto-char (point-min))
[-- Attachment #3: Type: text/plain, Size: 10 bytes --]
;; Colin
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch]
1999-01-04 19:40 [patch] Colin Rafferty
@ 1999-01-11 1:04 ` Lars Magne Ingebrigtsen
1999-01-11 1:46 ` [patch] Hrvoje Niksic
0 siblings, 1 reply; 5+ messages in thread
From: Lars Magne Ingebrigtsen @ 1999-01-11 1:04 UTC (permalink / raw)
Colin Rafferty <colin@xemacs.org> writes:
> Deleting a long line was being done incorrectly.
[...]
> - (delete-region (point) (progn (end-of-line) (point)))
> + (gnus-delete-line)
No, I've changed it so that long lines are shortened instead of
deleted. Is this a problem?
--
(domestic pets only, the antidote for overdose, milk.)
larsi@gnus.org * Lars Magne Ingebrigtsen
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch]
1999-01-11 1:04 ` [patch] Lars Magne Ingebrigtsen
@ 1999-01-11 1:46 ` Hrvoje Niksic
1999-01-11 7:01 ` [patch] Lars Magne Ingebrigtsen
0 siblings, 1 reply; 5+ messages in thread
From: Hrvoje Niksic @ 1999-01-11 1:46 UTC (permalink / raw)
Lars Magne Ingebrigtsen <larsi@gnus.org> writes:
> Colin Rafferty <colin@xemacs.org> writes:
>
> > Deleting a long line was being done incorrectly.
>
> [...]
>
> > - (delete-region (point) (progn (end-of-line) (point)))
> > + (gnus-delete-line)
>
> No, I've changed it so that long lines are shortened instead of
> deleted.
Is the "shortened" line saved to disk?
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch]
1999-01-11 1:46 ` [patch] Hrvoje Niksic
@ 1999-01-11 7:01 ` Lars Magne Ingebrigtsen
1999-01-11 15:54 ` [patch] Colin Rafferty
0 siblings, 1 reply; 5+ messages in thread
From: Lars Magne Ingebrigtsen @ 1999-01-11 7:01 UTC (permalink / raw)
Hrvoje Niksic <hniksic@srce.hr> writes:
> > No, I've changed it so that long lines are shortened instead of
> > deleted.
>
> Is the "shortened" line saved to disk?
No; the manipulation is only done to ease regexp comparison.
--
(domestic pets only, the antidote for overdose, milk.)
larsi@gnus.org * Lars Magne Ingebrigtsen
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch]
1999-01-11 7:01 ` [patch] Lars Magne Ingebrigtsen
@ 1999-01-11 15:54 ` Colin Rafferty
0 siblings, 0 replies; 5+ messages in thread
From: Colin Rafferty @ 1999-01-11 15:54 UTC (permalink / raw)
Lars Magne Ingebrigtsen writes:
> Hrvoje Niksic <hniksic@srce.hr> writes:
>> Lars Magne Ingebrigtsen writes:
>> > No, I've changed it so that long lines are shortened instead of
>> > deleted.
>> Is the "shortened" line saved to disk?
> No; the manipulation is only done to ease regexp comparison.
As long as I can see it when I read the message, this is not a
problem.
--
Colin
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~1999-01-11 15:54 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1999-01-04 19:40 [patch] Colin Rafferty
1999-01-11 1:04 ` [patch] Lars Magne Ingebrigtsen
1999-01-11 1:46 ` [patch] Hrvoje Niksic
1999-01-11 7:01 ` [patch] Lars Magne Ingebrigtsen
1999-01-11 15:54 ` [patch] Colin Rafferty
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).