all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#74705: [PATCH] Don't restrict 'lua-ts-send-file' to 'lua-ts-mode'
@ 2024-12-06  1:32 john muhl
  2024-12-06  1:35 ` john muhl
  0 siblings, 1 reply; 3+ messages in thread
From: john muhl @ 2024-12-06  1:32 UTC (permalink / raw)
  To: 74705

Tags: patch

The lua-ts-send-file command was marked as limited to lua-ts-mode
buffers but there is no reason for such a restriction since it
reads a file from disk (unlike the other send-* commands that use
buffer contents).





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#74705: [PATCH] Don't restrict 'lua-ts-send-file' to 'lua-ts-mode'
  2024-12-06  1:32 bug#74705: [PATCH] Don't restrict 'lua-ts-send-file' to 'lua-ts-mode' john muhl
@ 2024-12-06  1:35 ` john muhl
  2024-12-07 13:29   ` Eli Zaretskii
  0 siblings, 1 reply; 3+ messages in thread
From: john muhl @ 2024-12-06  1:35 UTC (permalink / raw)
  To: 74705

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: 0001-Don-t-restrict-lua-ts-send-file-to-lua-ts-mode.patch --]
[-- Type: text/x-patch, Size: 1019 bytes --]

From 097384e4ddaf98b1f2afe441d2c7997feb0bd715 Mon Sep 17 00:00:00 2001
From: john muhl <jm@pub.pink>
Date: Mon, 2 Dec 2024 09:20:01 -0600
Subject: [PATCH] Don't restrict 'lua-ts-send-file' to 'lua-ts-mode'

* lisp/progmodes/lua-ts-mode.el (lua-ts-send-file): Remove
restriction on interactive use.  Unlike related send-* commands
it can be useful to send a file to the Lua interpreter outside of
a 'lua-ts-mode' buffer.  (Bug#74705)
---
 lisp/progmodes/lua-ts-mode.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/progmodes/lua-ts-mode.el b/lisp/progmodes/lua-ts-mode.el
index 828636f359d..f7669a1b015 100644
--- a/lisp/progmodes/lua-ts-mode.el
+++ b/lisp/progmodes/lua-ts-mode.el
@@ -694,7 +694,7 @@ lua-ts-send-buffer
 
 (defun lua-ts-send-file (file)
   "Send contents of FILE to the inferior Lua process."
-  (interactive "f" lua-ts-mode)
+  (interactive "f")
   (with-temp-buffer
     (insert-file-contents-literally file)
     (lua-ts-send-region (point-min) (point-max))))
-- 
2.47.0


[-- Attachment #2: Type: text/plain, Size: 272 bytes --]



john muhl <jm@pub.pink> writes:

> Tags: patch
>
> The lua-ts-send-file command was marked as limited to lua-ts-mode
> buffers but there is no reason for such a restriction since it
> reads a file from disk (unlike the other send-* commands that use
> buffer contents).

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* bug#74705: [PATCH] Don't restrict 'lua-ts-send-file' to 'lua-ts-mode'
  2024-12-06  1:35 ` john muhl
@ 2024-12-07 13:29   ` Eli Zaretskii
  0 siblings, 0 replies; 3+ messages in thread
From: Eli Zaretskii @ 2024-12-07 13:29 UTC (permalink / raw)
  To: john muhl; +Cc: 74705-done

> From: john muhl <jm@pub.pink>
> Date: Thu, 05 Dec 2024 19:35:33 -0600
> 
> >From 097384e4ddaf98b1f2afe441d2c7997feb0bd715 Mon Sep 17 00:00:00 2001
> From: john muhl <jm@pub.pink>
> Date: Mon, 2 Dec 2024 09:20:01 -0600
> Subject: [PATCH] Don't restrict 'lua-ts-send-file' to 'lua-ts-mode'
> 
> * lisp/progmodes/lua-ts-mode.el (lua-ts-send-file): Remove
> restriction on interactive use.  Unlike related send-* commands
> it can be useful to send a file to the Lua interpreter outside of
> a 'lua-ts-mode' buffer.  (Bug#74705)
> ---
>  lisp/progmodes/lua-ts-mode.el | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lisp/progmodes/lua-ts-mode.el b/lisp/progmodes/lua-ts-mode.el
> index 828636f359d..f7669a1b015 100644
> --- a/lisp/progmodes/lua-ts-mode.el
> +++ b/lisp/progmodes/lua-ts-mode.el
> @@ -694,7 +694,7 @@ lua-ts-send-buffer
>  
>  (defun lua-ts-send-file (file)
>    "Send contents of FILE to the inferior Lua process."
> -  (interactive "f" lua-ts-mode)
> +  (interactive "f")
>    (with-temp-buffer
>      (insert-file-contents-literally file)
>      (lua-ts-send-region (point-min) (point-max))))
> -- 
> 2.47.0
> 
> john muhl <jm@pub.pink> writes:
> 
> > Tags: patch
> >
> > The lua-ts-send-file command was marked as limited to lua-ts-mode
> > buffers but there is no reason for such a restriction since it
> > reads a file from disk (unlike the other send-* commands that use
> > buffer contents).

Thanks, installed on the emacs-30 branch, and closing the bug.





^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-12-07 13:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-06  1:32 bug#74705: [PATCH] Don't restrict 'lua-ts-send-file' to 'lua-ts-mode' john muhl
2024-12-06  1:35 ` john muhl
2024-12-07 13:29   ` Eli Zaretskii

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.