Gnus development mailing list
 help / color / mirror / Atom feed
* 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).