all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Lars Ingebrigtsen <larsi@gnus.org>
To: Augusto Stoffel <arstoffel@gmail.com>
Cc: Andy Moreton <andrewjmoreton@gmail.com>,
	"Barton, Mark" <Mark.Barton@disney.com>,
	emacs-devel@gnu.org, Andreas Schwab <schwab@linux-m68k.org>,
	Stefan Monnier <monnier@iro.umontreal.ca>,
	49822@debbugs.gnu.org, Michael Albinus <michael.albinus@gmx.de>
Subject: bug#49822: master e32c7d2: Change Python eval to send directly instead of using temporary files
Date: Wed, 08 Sep 2021 09:50:22 +0200	[thread overview]
Message-ID: <877dfr5ww1.fsf__14960.0018792384$1631087529$gmane$org@gnus.org> (raw)
In-Reply-To: <875yvcmgm2.fsf@gmail.com> (Augusto Stoffel's message of "Tue, 07 Sep 2021 19:37:41 +0200")

Augusto Stoffel <arstoffel@gmail.com> writes:

> I have also rearranged things a bit so that the setup code is sent to
> the inferior process just once, rather than of on every call to
> `python-shell-send-string'.  This way, the smaller line length limit
> doesn't increase too much the use of temp files, which, as I mentioned,
> is slow over ssh.

This leads to compilation warnings about
python-shell-eval-file-setup-code etc.

Can you adjust the patch to fix that, and also add the defconst for
1024, like Eli discussed?

> PS: I have some more suggestions around the Python shell.  Is the ideal
> workflow to keep creating bugs with a small patch to each improvement,
> or do you prefer to review a larger collection of changes bundled
> together?

Smaller patches are fine by me, and you can submit them with `M-x
submit-emacs-patch'.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





  parent reply	other threads:[~2021-09-08  7:50 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20210903122828.16890.65271@vcs0.savannah.gnu.org>
     [not found] ` <20210903122829.EAAC220B71@vcs0.savannah.gnu.org>
2021-09-03 23:04   ` master e32c7d2: Change Python eval to send directly instead of using temporary files Stefan Monnier
2021-09-04  9:49     ` bug#49822: " Augusto Stoffel
2021-09-04 13:52       ` Stefan Monnier
2021-09-04 13:52       ` bug#49822: " Stefan Monnier via Bug reports for GNU Emacs, the Swiss army knife of text editors
2021-09-05  6:13       ` Barton, Mark
2021-09-05  7:46         ` bug#49822: " Augusto Stoffel
2021-09-05  8:10         ` Augusto Stoffel
2021-09-05 17:18           ` bug#49822: " Mark Barton
2021-09-05 17:33             ` Mark Barton
2021-09-05 17:46             ` Augusto Stoffel
2021-09-05 17:46             ` bug#49822: " Augusto Stoffel
2021-09-05  8:10         ` Augusto Stoffel
2021-09-05  6:13       ` Barton, Mark
2021-09-05  7:41       ` bug#32042: " Lars Ingebrigtsen
2021-09-05  7:41       ` Lars Ingebrigtsen
2021-09-05  7:41       ` Lars Ingebrigtsen
2021-09-05 16:36       ` Andreas Schwab
2021-09-05 18:40         ` Augusto Stoffel
2021-09-06  7:43           ` Michael Albinus
2021-09-06  7:43           ` Michael Albinus
2021-09-06  8:40             ` Andreas Schwab
2021-09-06 11:23               ` Michael Albinus
2021-09-06 11:53                 ` Andreas Schwab
2021-08-02 14:32                   ` bug#49822: 28.0.50; python-shell-send functions show no output dalanicolai
2021-08-02 15:11                     ` bug#49822: forgot link (and mentioned wrong user) dalanicolai
2021-08-02 15:17                     ` bug#49822: 28.0.50; python-shell-send functions show no output Andrea Corallo via Bug reports for GNU Emacs, the Swiss army knife of text editors
2021-08-04  7:18                     ` Lars Ingebrigtsen
2021-08-05 10:03                       ` dalanicolai
2021-08-12 14:57                       ` Augusto Stoffel
2021-08-13 11:10                         ` Lars Ingebrigtsen
2021-08-13 16:44                           ` Andreas Röhler
2021-08-14  9:29                             ` Augusto Stoffel
2021-08-15  6:41                               ` Andreas Röhler
2021-08-26  8:27                                 ` Augusto Stoffel
2021-08-26 14:09                                   ` Lars Ingebrigtsen
2021-08-28  9:28                                     ` bug#49822: [RFC PATCH] " Augusto Stoffel
2021-08-28 15:41                                       ` Lars Ingebrigtsen
2021-09-03  8:02                                         ` bug#49822: [PATCH] " Augusto Stoffel
2021-09-03 12:28                                           ` Lars Ingebrigtsen
2021-09-06 22:15                     ` bug#49822: master e32c7d2: Change Python eval to send directly instead of using temporary files Andy Moreton
2021-09-07  7:18                       ` Andreas Schwab
2021-09-06 12:00                   ` Michael Albinus
2021-09-06 16:08                     ` Augusto Stoffel
2021-09-06 16:08                     ` Augusto Stoffel
2021-09-07 17:37                   ` Augusto Stoffel
2021-09-07 17:48                     ` Eli Zaretskii
2021-09-07 17:59                       ` Augusto Stoffel
2021-09-07 17:59                       ` Augusto Stoffel
2021-09-07 18:19                         ` Eli Zaretskii
2021-09-07 18:13                       ` Augusto Stoffel
2021-09-07 18:31                         ` Eli Zaretskii
2021-09-07 19:00                           ` Augusto Stoffel
2021-09-07 19:16                             ` Eli Zaretskii
2021-09-08  7:02                               ` Michael Albinus
2021-09-08  7:02                               ` Michael Albinus
2021-09-07 19:16                             ` Eli Zaretskii
2021-09-07 18:31                         ` Eli Zaretskii
2021-09-07 18:13                       ` Augusto Stoffel
2021-09-07 17:48                     ` Eli Zaretskii
2021-09-08  3:17                     ` Barton, Mark
2021-09-08  5:09                       ` Augusto Stoffel
2021-09-08  7:50                     ` Lars Ingebrigtsen [this message]
2021-09-08  7:50                     ` Lars Ingebrigtsen
2021-09-08 14:05                       ` Augusto Stoffel
2021-09-08 14:05                       ` Augusto Stoffel
2021-09-09 13:48                         ` Lars Ingebrigtsen
2021-09-06 11:23               ` Michael Albinus
2021-09-06  8:40             ` Andreas Schwab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='877dfr5ww1.fsf__14960.0018792384$1631087529$gmane$org@gnus.org' \
    --to=larsi@gnus.org \
    --cc=49822@debbugs.gnu.org \
    --cc=Mark.Barton@disney.com \
    --cc=andrewjmoreton@gmail.com \
    --cc=arstoffel@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=michael.albinus@gmx.de \
    --cc=monnier@iro.umontreal.ca \
    --cc=schwab@linux-m68k.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.