Project

General

Profile

Bug #772

Emacs UI: send-region and send-line make sense inside a cpkg5 file?

Added by John Abbott over 8 years ago. Updated over 8 years ago.

Status:
In Progress
Priority:
Low
Assignee:
-
Category:
EmacsUI
Target version:
Start date:
05 Sep 2015
Due date:
% Done:

20%

Estimated time:
Spent time:

Description

Does it make sense to have send-region and send-line easily accessible from inside a file.cpkg5? Sending just part of the file without the proper package context actually changes slightly the meaning of that excerpt!

History

#1 Updated by Anna Maria Bigatti over 8 years ago

  • % Done changed from 0 to 20

John Abbott wrote:

Does it make sense to have send-region and send-line easily accessible from inside a file.cpkg5? Sending just part of the file without the proper package context actually changes slightly the meaning of that excerpt!

I think so. For many reasons:
0 - it is much easier for the emacs-developer to leave it like that ;-)
1 - it might indeed be quite tricky to implement the restriction
2 - it is simpler for a user to have identical settings in .cocoa5 and .cpkg
3 - a package developer should know what he's doing (maybe testing a small example temporarily written in the package..)

#2 Updated by John Abbott over 8 years ago

  • Status changed from New to In Progress
  • Priority changed from Normal to Low
  • Target version changed from CoCoA-5.1.3/4 Jan 2016 to CoCoA-5.?.?

JAA continues to be unconvinced.

Point (0): it is correct (and will probably prevail).
Point (1): it will probably be somewhat tricky, but not impossible.
Point (2): but they're not "identical" since they don't really do the same thing!
Point (3): this probably true.

It might be possible for send-region and send-line to pop-up a warning that the code would be interpreted outside the package context (and perhaps ask the user for confirmation?)... this could be "tricky" to implement (see point (1)).

It's not that important: JAA changed priority to "low", and target to "far away".

Also available in: Atom PDF