unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* info.el with auto-compression-mode
@ 2005-11-27  3:42 Yoichi NAKAYAMA
  2005-11-28  4:46 ` Richard M. Stallman
  0 siblings, 1 reply; 3+ messages in thread
From: Yoichi NAKAYAMA @ 2005-11-27  3:42 UTC (permalink / raw)


After the split of jka-compr, info-mode cannot use
auto-compression-mode if jka-compr is not loaded.
Following patch will fix this.

--- info.el	25 Nov 2005 16:35:34 -0000	1.462
+++ info.el	27 Nov 2005 02:53:53 -0000
@@ -35,7 +35,7 @@
 
 ;;; Code:
 
-(eval-when-compile (require 'jka-compr))
+(eval-when-compile (require 'jka-cmpr-hook))
 
 (defgroup info nil
   "Info subsystem."
@@ -448,7 +448,7 @@
       (or tail
 	  (error "Can't find %s or any compressed version of it" filename)))
     ;; check for conflict with jka-compr
-    (if (and (featurep 'jka-compr)
+    (if (and (featurep 'jka-cmpr-hook)
 	     (jka-compr-installed-p)
 	     (jka-compr-get-compression-info fullname))
 	(setq decoder nil))


Regards,
-- 
Yoichi NAKAYAMA

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: info.el with auto-compression-mode
  2005-11-27  3:42 info.el with auto-compression-mode Yoichi NAKAYAMA
@ 2005-11-28  4:46 ` Richard M. Stallman
  2005-11-28 11:26   ` Yoichi NAKAYAMA
  0 siblings, 1 reply; 3+ messages in thread
From: Richard M. Stallman @ 2005-11-28  4:46 UTC (permalink / raw)
  Cc: emacs-devel

    -(eval-when-compile (require 'jka-compr))
    +(eval-when-compile (require 'jka-cmpr-hook))

That makes no sense.  jka-compr-hook is always loaded.
Does this patch give good results?


*** info.el	26 Nov 2005 20:39:47 -0500	1.462
--- info.el	27 Nov 2005 22:20:41 -0500	
***************
*** 448,455 ****
        (or tail
  	  (error "Can't find %s or any compressed version of it" filename)))
      ;; check for conflict with jka-compr
!     (if (and (featurep 'jka-compr)
! 	     (jka-compr-installed-p)
  	     (jka-compr-get-compression-info fullname))
  	(setq decoder nil))
      (if decoder
--- 448,454 ----
        (or tail
  	  (error "Can't find %s or any compressed version of it" filename)))
      ;; check for conflict with jka-compr
!     (if (and (jka-compr-installed-p)
  	     (jka-compr-get-compression-info fullname))
  	(setq decoder nil))
      (if decoder

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: info.el with auto-compression-mode
  2005-11-28  4:46 ` Richard M. Stallman
@ 2005-11-28 11:26   ` Yoichi NAKAYAMA
  0 siblings, 0 replies; 3+ messages in thread
From: Yoichi NAKAYAMA @ 2005-11-28 11:26 UTC (permalink / raw)
  Cc: emacs-devel

At Sun, 27 Nov 2005 23:46:33 -0500,
Richard M. Stallman wrote:
> Does this patch give good results?
> 
> 
> *** info.el	26 Nov 2005 20:39:47 -0500	1.462
> --- info.el	27 Nov 2005 22:20:41 -0500	
> ***************
> *** 448,455 ****
>         (or tail
>   	  (error "Can't find %s or any compressed version of it" filename)))
>       ;; check for conflict with jka-compr
> !     (if (and (featurep 'jka-compr)
> ! 	     (jka-compr-installed-p)
>   	     (jka-compr-get-compression-info fullname))
>   	(setq decoder nil))
>       (if decoder
> --- 448,454 ----
>         (or tail
>   	  (error "Can't find %s or any compressed version of it" filename)))
>       ;; check for conflict with jka-compr
> !     (if (and (jka-compr-installed-p)
>   	     (jka-compr-get-compression-info fullname))
>   	(setq decoder nil))
>       (if decoder

Yes, it works.
-- 
Yoichi NAKAYAMA

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2005-11-28 11:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-11-27  3:42 info.el with auto-compression-mode Yoichi NAKAYAMA
2005-11-28  4:46 ` Richard M. Stallman
2005-11-28 11:26   ` Yoichi NAKAYAMA

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).