Re: gnus-article-hide-headers beeps: Buffer is read-only

[Reiner Steib <reinersteib+from-uce@imap.cc>]

On Sat, Aug 04 2007, Sven Joachim wrote:

> Gnus v5.11
> GNU Emacs 22.1.50.1 (i486-pc-linux-gnu, GTK+ Version 2.10.13)
>  of 2007-07-31 on debian, modified by Debian
> 200 news.gmane.org InterNetNews NNRP server INN 2.4.1 ready (posting ok).
>
> When typing "W W h" in a *Summary* buffer, Emacs beeps and displays
> the error message: "Buffer is read-only: #<buffer *Article*>" (in
> *Messages* this shows up as
> "article-hide-headers: Buffer is read-only: #<buffer *Article*>").
>
> Enabling debug-on-error does not enter the debugger in this case.

Thanks for your report.  Does the following patch fix it?

--8<---------------cut here---------------start------------->8---
--- gnus-art.el	04 Aug 2007 20:35:29 +0200	1.125.2.5
+++ gnus-art.el	04 Aug 2007 22:14:56 +0200	
@@ -1743,7 +1743,7 @@
   (interactive)
   ;; This function might be inhibited.
   (unless gnus-inhibit-hiding
-    (let ((inhibit-read-only nil)
+    (let ((inhibit-read-only t)
 	  (case-fold-search t)
 	  (max (1+ (length gnus-sorted-header-list)))
 	  (inhibit-point-motion-hooks t)
--8<---------------cut here---------------end--------------->8---

Katsumi, the problem was introduced by this change ...

--8<---------------cut here---------------start------------->8---
revision 6.381.2.6
date: 2004/08/26 09:59:42;  author: yamaoka;  state: Exp;  lines: +55 -47
[...]
* gnus-art.el (article-hide-headers): Refer to the values for
gnus-ignored-headers and gnus-visible-headers in the summary
buffer since a user may have set them as group parameters.
(gnus-article-read-summary-keys): Restore new window-start and
hscroll to summary window.
(gnus-prev-page-map): Remove duplicated one.
--8<---------------cut here---------------end--------------->8---

... and fixed in the trunk by Lars:

--8<---------------cut here---------------start------------->8---
revision 7.185
date: 2006/06/21 18:14:23;  author: larsi;  state: Exp;  lines: +1 -1
* gnus-art.el (article-hide-headers): Inhibit read-only stuff.
--8<---------------cut here---------------end--------------->8---

I think the wrong value of inhibit-read-only was just a mistake
(replacing "buffer-read-only" with "inhibit-read-only"?).  But the
ChangeLog doesn't say anything about removing (save-excursion
(save-restriction ...)).

I doubt the other occurrence of "(inhibit-read-only nil)" in
`gnus-x-splash' is correct.  Is it?

--8<---------------cut here---------------start------------->8---
revision 7.17
date: 2007/06/07 11:59:35;  author: yamaoka;  state: Exp;  lines: +79 -34
* gnus-ems.el (gnus-x-splash): Make it work.
--8<---------------cut here---------------end--------------->8---

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/