I noticed that my writing was a little screwed up earlier.  

I meant to say that the first "fix" dose *not* work with all cases.  I like the first 
approach better but since it does not work, I took the second less ideal one.  

I guess the reason for the first one not to work is because the recursive nature of 
the function.  By the time we really need the "old seperator", we have lost it already.

Since the rest of the code in the jde.el is using the system-type variable, I thought 
it would be safe to use it.  I don't have the cygwin version installed but I am under 
the impression that cygwin wouldn't need to use ?\\.  I could be wrong.  If you have 
that installed, could you please let me know if the directory paths in the buffer list 
(C-x C-b) are using "/" or "\" as the seperator.  If it is "/" than we should be OK, 
otherwise, not.

I have never needed to deal with Windows/Unix issues before, so I am not familiar with 
directory-sep-char.  I guess that there must be a better way to deal with this if it 
is to be deprecated.

Thanks.

Ping Liang


Jason Rumney <[EMAIL PROTECTED]> wrote:

>[EMAIL PROTECTED] (Ping Liang) writes:
>
>>     (let ((directory-sep-char ?/)) ;; Ping's fix
>>       (if (eq system-type 'windows-nt) ;; Ping's fix
>>           (setq directory-sep-char ?\\)) ;; Ping's fix
>>       (expand-file-name file dir))
>
>I don't understand why this is necessary.  If XEmacs only accepts \ as
>directory separator in expand-file-name, then your previous patch
>(which set it back to its previous value after the code that required
>it to be /) should work.
>
>The above code looks like it will do the wrong thing on
>the Cygwin version of XEmacs, and although the NT port of GNU Emacs
>will work with the above code, directory-sep-char comes with the
>following warning:
>
>WARNING: This variable is deprecated and will be removed in the near
>future.  DO NOT USE IT.
>
>
>-- 
>Jason Rumney
>
>
-- 




__________________________________________________________________
Your favorite stores, helpful shopping tools and great gift ideas. Experience the 
convenience of buying online with Shop@Netscape! http://shopnow.netscape.com/

Get your own FREE, personal Netscape Mail account today at http://webmail.netscape.com/

Reply via email to