* gnus-summary-end-of-article displays beginning of article
@ 2008-03-03 18:49 Sven Joachim
2008-03-04 22:31 ` Reiner Steib
0 siblings, 1 reply; 5+ messages in thread
From: Sven Joachim @ 2008-03-03 18:49 UTC (permalink / raw)
To: ding
With current No Gnus CVS I'm noticing the unwanted behavior that
gnus-summary-end-of-article goes to the beginning of the article, rather
than to the end. Anyone else seeing this?
I've tried to single-step that function, but failed to find anything
obviously wrong. And it had not been changed for ages.
Sven
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: gnus-summary-end-of-article displays beginning of article
2008-03-03 18:49 gnus-summary-end-of-article displays beginning of article Sven Joachim
@ 2008-03-04 22:31 ` Reiner Steib
2008-03-06 8:25 ` Sven Joachim
0 siblings, 1 reply; 5+ messages in thread
From: Reiner Steib @ 2008-03-04 22:31 UTC (permalink / raw)
To: ding
On Mon, Mar 03 2008, Sven Joachim wrote:
> With current No Gnus CVS I'm noticing the unwanted behavior that
> gnus-summary-end-of-article goes to the beginning of the article, rather
> than to the end. Anyone else seeing this?
>
> I've tried to single-step that function, but failed to find anything
> obviously wrong. And it had not been changed for ages.
`gnus-summary-end-of-article' has been modified in 5.10.3:
,----
| 2003-10-21 Katsumi Yamaoka <yamaoka@jpl.org>
|
| * gnus-art.el (gnus-narrow-to-page): Clear as well as set the
| value for gnus-page-broken.
|
| * gnus-sum.el (gnus-summary-beginning-of-article): Use
| gnus-break-pages instead of gnus-page-broken.
| (gnus-summary-end-of-article): Use gnus-break-pages instead of
| gnus-page-broken; narrow to the end of a page beforehand.
| (gnus-summary-toggle-header): Use gnus-break-pages instead of
| gnus-page-broken; remove delimiter buttons unless gnus-break-pages
| is non-nil.
`----
After reverting the following hunk, it works as expected for me.
,----
| --- gnus-sum.el 19 Oct 2003 01:09:52 -0000 6.369
| +++ gnus-sum.el 21 Oct 2003 14:28:32 -0000 6.370
| [...]
| @@ -8518,7 +8518,9 @@
| (widen)
| (goto-char (point-max))
| (recenter -3)
| - (when gnus-page-broken
| + (when gnus-break-pages
| + (when (re-search-backward page-delimiter nil t)
| + (narrow-to-region (match-end 0) (point-max)))
| (gnus-narrow-to-page))))
|
| (defun gnus-summary-print-truncate-and-quote (string &optional len)
`----
Bye, Reiner.
--
,,,
(o o)
---ooO-(_)-Ooo--- | PGP key available | http://rsteib.home.pages.de/
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: gnus-summary-end-of-article displays beginning of article
2008-03-04 22:31 ` Reiner Steib
@ 2008-03-06 8:25 ` Sven Joachim
2008-03-06 8:45 ` Katsumi Yamaoka
0 siblings, 1 reply; 5+ messages in thread
From: Sven Joachim @ 2008-03-06 8:25 UTC (permalink / raw)
To: ding
On 2008-03-04 23:31 +0100, Reiner Steib wrote:
> On Mon, Mar 03 2008, Sven Joachim wrote:
>
>> With current No Gnus CVS I'm noticing the unwanted behavior that
>> gnus-summary-end-of-article goes to the beginning of the article, rather
>> than to the end. Anyone else seeing this?
>>
>> I've tried to single-step that function, but failed to find anything
>> obviously wrong. And it had not been changed for ages.
>
> `gnus-summary-end-of-article' has been modified in 5.10.3:
That's what I meant with "ages". And it still works fine in current
Gnus 5.11 from EMACS_22_BASE.
> After reverting the following hunk, it works as expected for me.
>
> ,----
> | --- gnus-sum.el 19 Oct 2003 01:09:52 -0000 6.369
> | +++ gnus-sum.el 21 Oct 2003 14:28:32 -0000 6.370
> | [...]
> | @@ -8518,7 +8518,9 @@
> | (widen)
> | (goto-char (point-max))
> | (recenter -3)
> | - (when gnus-page-broken
> | + (when gnus-break-pages
> | + (when (re-search-backward page-delimiter nil t)
> | + (narrow-to-region (match-end 0) (point-max)))
> | (gnus-narrow-to-page))))
> |
> | (defun gnus-summary-print-truncate-and-quote (string &optional len)
> `----
Okay, for the moment I have set gnus-break-pages to nil, which works
around the problem.
Still puzzled,
Sven
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: gnus-summary-end-of-article displays beginning of article
2008-03-06 8:25 ` Sven Joachim
@ 2008-03-06 8:45 ` Katsumi Yamaoka
2008-03-07 6:24 ` Katsumi Yamaoka
0 siblings, 1 reply; 5+ messages in thread
From: Katsumi Yamaoka @ 2008-03-06 8:45 UTC (permalink / raw)
To: ding
>>>>> Reiner Steib wrote:
> `gnus-summary-end-of-article' has been modified in 5.10.3:
>| 2003-10-21 Katsumi Yamaoka <yamaoka@jpl.org>
> After reverting the following hunk, it works as expected for me.
[...]
>>>>> Sven Joachim wrote:
> Okay, for the moment I have set gnus-break-pages to nil, which works
> around the problem.
Sorry for inconvenience. Please let me have time to fix it.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: gnus-summary-end-of-article displays beginning of article
2008-03-06 8:45 ` Katsumi Yamaoka
@ 2008-03-07 6:24 ` Katsumi Yamaoka
0 siblings, 0 replies; 5+ messages in thread
From: Katsumi Yamaoka @ 2008-03-07 6:24 UTC (permalink / raw)
To: ding
>>>>> Katsumi Yamaoka wrote:
>>>>>> Sven Joachim wrote:
>> Okay, for the moment I have set gnus-break-pages to nil, which works
>> around the problem.
> Sorry for inconvenience. Please let me have time to fix it.
Fixed in the Gnus trunk and the v5-10 branch. Although I didn't
see why `gnus-summary-end-of-article' worked in Emacs 21 and
XEmacs, I also realized some other related functions were not
going well. Thanks for pointing it out anyway.
Regards,
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2008-03-07 6:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-03-03 18:49 gnus-summary-end-of-article displays beginning of article Sven Joachim
2008-03-04 22:31 ` Reiner Steib
2008-03-06 8:25 ` Sven Joachim
2008-03-06 8:45 ` Katsumi Yamaoka
2008-03-07 6:24 ` Katsumi Yamaoka
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).