Re: completer.el

Luke Gorrie <[email protected]> writes:

> First, I grabbed the second-newest revision (1.4). The latest adds
> this:
>
>   +;; Necessary when loading completer.el, but have not yet loaded ilisp
>   +(require 'ilcompat) 
>
> So version 1.4 is stand-alone, but 1.5 depends on another small but
> ILISP-specific file. What it seems to want is the definition of
> +ilisp-emacs-version+ for this code:
>
>   (not (memq +ilisp-emacs-version-id+
>        '(xemacs lucid-19 lucid-19-new)))
>
> But in my copy I replaced this with (not (featurep 'xemacs)), just to
> cut the dependency.
>
> Any comments? I don't want installing SLIME to somehow break ILISP.

Looks ok to me.  I think the same modification should be done to
the ILISP version of it as well.  It seems to me that the ILISP
project is The Place where completer.el is "maintained" these days,
so it would be good if it stayed independent of ILISP so that it
could be used stand-alone.

-- 
Hannu


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/