RE: [PATCH] jde-help-show-class-member-doc doesn't work on Xemacs

2001-08-08 Thread Nick Sieger 

I believe IE understands the 'file:///D|/...' notation also, so perhaps that
should become the standard on windows to convert 'drive:' to 'drive|' when
building the URL.

/Nick

-Original Message-
From: Dr. Volker Zell [mailto:[EMAIL PROTECTED]]
Sent: Wednesday, August 08, 2001 12:07 PM
To: Paul Kinnucan
Cc: jde-list
Subject: [PATCH] jde-help-show-class-member-doc doesn't work on Xemacs

[NJS: patch omitted]

But a problem remains. At least on my system. I'm using Windows NT with
Netscape.
Netscape does not seem to accept file URL's of the form you are writing to
the
temporary html file.

For example:

In my case

file:///java/api/java/lang/Object.html#toString()

must be set to

file:///D|/java/api/java/lang/Object.html#toString()
^^
to be seen inside Netscape. Notice the drive letter D: in the file URL.
This is where I have my api docu.

Should we create a customizable variable and concat together the
file:/// + var + /path/to/api-docu ??

Ciao
  Volker

Re: [PATCH] jde-help-show-class-member-doc doesn't work on Xemacs

2001-08-08 Thread Paul Kinnucan 

At 07:07 PM 8/8/2001 +0200, you wrote:
 Paul == Paul Kinnucan [EMAIL PROTECTED] writes:

Paul http://jde.sunsite.dk/
Paul JDE 2.2.8beta5

Hi Paul

This fixes jde-help-show-class-member-doc because temporary-file-directory is
not defined in Xemacs.

-- cut here 
-

diff -c /usr/local/lib/xemacs/site-packages/lisp/jde/jde-help.el~ 
/usr/local/lib/xemacs/site-packages/lisp/jde/jde-help.el
*** /usr/local/lib/xemacs/site-packages/lisp/jde/jde-help.el~  Wed Aug  8 
18:52:46 2001
--- /usr/local/lib/xemacs/site-packages/lisp/jde/jde-help.el   Wed Aug  8 
18:52:46 2001
***
*** 251,257 
  
  
  (defun jde-help-show-class-member-doc (docfile)
!   (let* ((metafile (expand-file-name jde_meta.html 
temporary-file-directory))
   (buff (find-file-noselect metafile))
   (standard-output buff))
  (if buff
--- 251,260 
  
  
  (defun jde-help-show-class-member-doc (docfile)
!   (let* ((temp-directory
!(or (if (fboundp 'temp-directory) (temp-directory))
!(if (boundp 'temporary-file-directory) temporary-file-directory)))
!  (metafile (expand-file-name jde_meta.html temp-directory))
   (buff (find-file-noselect metafile))
   (standard-output buff))
  (if buff

-- cut here 
-


I have updated the CVS repository with this fix.

But a problem remains. At least on my system. I'm using Windows NT with 
Netscape.
Netscape does not seem to accept file URL's of the form you are writing to the
temporary html file.

For example:

In my case

file:///java/api/java/lang/Object.html#toString()

must be set to

file:///D|/java/api/java/lang/Object.html#toString()
^^
to be seen inside Netscape. Notice the drive letter D: in the file URL.
This is where I have my api docu.

Should we create a customizable variable and concat together the
file:/// + var + /path/to/api-docu ??


The latest version of jde-help.el now replaces the colon in the drive
prefix in DOS paths with a vertical bar when forming a URL.

- Paul