From xemacs-m  Sun May 18 20:11:41 1997
Received: from mercury.Sun.COM (mercury.Sun.COM [192.9.25.1])
	by xemacs.org (8.8.5/8.8.5) with SMTP id UAA15026
	for <xemacs-beta@xemacs.org>; Sun, 18 May 1997 20:11:40 -0500 (CDT)
Received: from Eng.Sun.COM ([129.146.1.25]) by mercury.Sun.COM (SMI-8.6/mail.byaddr) with SMTP id SAA21262 for <xemacs-beta@xemacs.org>; Sun, 18 May 1997 18:25:47 -0700
Received: from kindra.eng.sun.com by Eng.Sun.COM (SMI-8.6/SMI-5.3)
	id SAA24895; Sun, 18 May 1997 18:11:03 -0700
Received: from xemacs.eng.sun.com by kindra.eng.sun.com (SMI-8.6/SMI-SVR4)
	id SAA04496; Sun, 18 May 1997 18:11:06 -0700
Received: by xemacs.eng.sun.com (SMI-8.6/SMI-SVR4)
	id SAA10058; Sun, 18 May 1997 18:11:02 -0700
Date: Sun, 18 May 1997 18:11:02 -0700
Message-Id: <199705190111.SAA10058@xemacs.eng.sun.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
From: Martin Buchholz <mrb@Eng.Sun.COM>
To: Hrvoje Niksic <hniksic@srce.hr>
Cc: XEmacs Developers <xemacs-beta@xemacs.org>
Subject: Re: ediff problems in 20.2-b6 (It worked with 20.2-b4)
In-Reply-To: <kig67whxngl.fsf@jagor.srce.hr>
References: <199705180545.IAA13640@merlin.ornet.co.il>
	<kig67whxngl.fsf@jagor.srce.hr>
X-Mailer: VM 6.31 under 20.2 XEmacs Lucid
Reply-To: Martin Buchholz <mrb@Eng.Sun.COM>

>>>>> "Hrv" == Hrvoje Niksic <hniksic@srce.hr> writes:

Hrv> Ricardo Marek <ricky@ornet.co.il> writes:
>> After loading the two files with XEmacs 20.2-b6,
>> I run ediff and get:
>> 
>> Wrong type argument: stringp, nil

Hrv> Ricardo, whenever you get an error that makes Emacs go "beep", try
Hrv> setting `debug-on-error' to t (e.g. via `M-x customize-variable' :-)
Hrv> and send us the backtrace you get.

Hrv> Off the top of my mind, I think this might be related to having a `::' 
Hrv> in your PATH.  BTW, does anyone know *why* is `parse-colon-path' such
Hrv> an abomination?

Hrv> (parse-colon-path "ab::cd:e:")
Hrv>   => ("ab/" nil "cd/" "e/" nil)

Hrv> What's wrong with:

Hrv> (defun my-parse-colon-path (path)
Hrv>   (split-string path ":"))

Hrv> (my-parse-colon-path "ab::cd:e:")
Hrv>   => ("ab" "" "cd" "e" "")

Having been horribly burned myself by the nil's generated by the
current behavior, I would vote to change to Hrvoje's behavior,
backward-incompatible as it might be.  But first we ought to convince
RMS.  Hrvoje, could you try to do this?  While you're at it, you could
convince him to add (split-string) and (locate-file), two functions
whose glaring absence has annoyed me recently.

Martin

