emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: New feature in project.el: Remembering the previously used projects


From: Basil L. Contovounesios
Subject: Re: New feature in project.el: Remembering the previously used projects
Date: Fri, 29 May 2020 00:05:12 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)

Dmitry Gutov <dgutov@yandex.ru> writes:

> Thanks some good work by Simen, project commands (project-find-file, etc) now
> prompt you to choose among known projects, if currently outside of all project
> directories.
>
> It's now on master.
>
> There's also a new command, project-switch-project.

Thanks!

> Please let us know how you like the change, and if you see any new problems.

Only one feature request and some minor questions from me.

Can the project-list file name please be customisable?

Could the contents of the project-list file comprise easily readable,
printable, and even extensible sexps?

Could project-switch-project reuse read-multiple-choice or similar?

Isn't there a race here?

  (defun project--ensure-file-exists (filename)
    "Create an empty file FILENAME if it doesn't exist."
    (unless (file-exists-p filename)
      (with-temp-buffer
        (write-file filename))))

Why not use something like the following instead?

  (ignore-error file-already-exists
    (write-region "" nil filename nil 0 nil 'excl))

In fact, couldn't project--ensure-file-exists be eliminated altogether?
If the file doesn't exist, just don't set project--list, or set it to
nil.

Thanks,

-- 
Basil



reply via email to

[Prev in Thread] Current Thread [Next in Thread]