-
-
Notifications
You must be signed in to change notification settings - Fork 29
Description
I propose removing the "Lisp Edit" sub-item from the "Edit" menu entry of FILEBROWSER.
Currently, what this does, is to, check if it is LISPSOURCEFILEP, if so, it checks if it is already loaded and the current version (by comparing file paths). If it isn't, then it asks to load the file PROP.
If it is already loaded and current, or it just did the LOAD PROP, it opens the editor on the fileCOMS. (I.e., it does (ED rootOfFilename '(FILES :DONTWAIT))
There may be some cases where this is useful, but very few at most. And there are ways that the implementation of this process can fail its apparent intent.
In addition to removing the "Lisp Edit" sub-item, (which makes the "Tedit" sub-item redundant, so remove that also), I suggest in the Tedit case, perform a LISPSOURCEFILEP check and prompt for user confirmation with a message like: "Directly editing a Medley lisp source file is not recommended. Are you sure?"
Metadata
Metadata
Assignees
Labels
Type
Projects
Status