[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] scratch/project 106e023: Add new `project' package, an
From: |
Stefan Monnier |
Subject: |
Re: [Emacs-diffs] scratch/project 106e023: Add new `project' package, and use it in xref |
Date: |
Wed, 08 Jul 2015 10:06:57 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) |
> Sounds good. Please take a look again.
Good, thanks.
>> Why use a cons cell with the car's symbol standing for a type tag, when
>> the carried object is already properly self-describing (IIUC it's an
>> EIEIO object)?
> Because then dispatching any `project-root' call fails at runtime if ede is
> not loaded. And I don't want to make it a runtime dependency.
> This function should probably be in ede.el, and only added to
> project-find-functions when it's loaded.
Exactly: rather than work around the problem by using a cons cell,
better do it by moving the function where it belongs.
> Any comments on the "source directories" issue? I'm inclining towards
> having a project-source-directories-function variable now, which in
> emacs-lisp-mode would return load-path.
I haven't had time to look into that yet, sorry.
Stefan