Gnus development mailing list
 help / color / mirror / Atom feed
From: Reiner Steib <reinersteib+gmane@imap.cc>
To: ding@gnus.org
Subject: Re: gnus-summary-end-of-article displays beginning of article
Date: Tue, 04 Mar 2008 23:31:43 +0100	[thread overview]
Message-ID: <v9ableazow.fsf@marauder.physik.uni-ulm.de> (raw)
In-Reply-To: <87d4qblk24.fsf@gmx.de> (Sven Joachim's message of "Mon, 03 Mar 2008 19:49:23 +0100")

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/



  reply	other threads:[~2008-03-04 22:31 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-03 18:49 Sven Joachim
2008-03-04 22:31 ` Reiner Steib [this message]
2008-03-06  8:25   ` Sven Joachim
2008-03-06  8:45     ` Katsumi Yamaoka
2008-03-07  6:24       ` Katsumi Yamaoka

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