From: Stefan Kangas <stefankangas@gmail.com>
To: Jim Porter <jporterbugs@gmail.com>
Cc: Morgan Smith <Morgan.J.Smith@outlook.com>,
Lars Ingebrigtsen <larsi@gnus.org>,
57367@debbugs.gnu.org
Subject: bug#57367: [PATCH] Speed up em-smart
Date: Wed, 6 Sep 2023 15:46:28 -0700 [thread overview]
Message-ID: <CADwFkmnC8Y6kUHFDM5Lp3C2ZpieUejpAdMj_gFRY=q+p0TEAFA@mail.gmail.com> (raw)
In-Reply-To: <f38f8ebb-9d3a-5ea7-2c01-6ab57f514fe3@gmail.com> (Jim Porter's message of "Thu, 8 Sep 2022 21:36:17 -0700")
Jim Porter <jporterbugs@gmail.com> writes:
> On 9/6/2022 6:30 PM, Morgan Smith wrote:
>> I've attached my patch V2.
>> This restores some more of the original logic that I have now realized
>> was indeed necessary. Again, this patch should not actually change
>> anything with respect to program logic, flow, or user experience. In my
>> limited testing, it seems to act just as it did before (but
>> significantly more performant).
>
> Thanks, this does indeed seem a lot faster. I do notice one small change in
> behavior though: after starting Eshell with the em-smart module loaded, run
> "echo hi" and then split the window vertically with 'C-x 2'. Before the patch,
> the window doesn't scroll. After the patch, the window is scrolled so that the
> "echo hi" block is at the top (i.e. the "Welcome to the Emacs shell" banner is
> hidden).
>
> The same sort of thing happens if you run a command with a lot of output first
> and then run "echo hi". Before splitting, the "echo hi" block is at the
> bottom. Without this patch, 'C-x 2' maintains that position (i.e. it scrolls the
> minimum amount of the previous command out of view so that you can see the
> most-recent command). With the patch, 'C-x 2' scrolls *all* of the previous
> command out of view, so the "echo hi" block is at the top.
>
> I think the pre-patch behavior is the most-usable: if you can fit all of the
> last command in the window, it should be at the bottom so that you can see (some
> of) the previous command.
>
> (I also saw some strange issue with resizing the windows via the mouse, but I
> can't reproduce it anymore. If I can figure out how trigger this reliably, I'll
> send an update. Sorry that's not very helpful at present...)
That was one year ago.
Morgan, did you get a chance to look into the issues Jim pointed out
above?
next prev parent reply other threads:[~2023-09-06 22:46 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-23 20:06 bug#57367: [PATCH] Speed up em-smart Morgan Smith
2022-09-04 21:56 ` Lars Ingebrigtsen
2022-09-05 19:01 ` Jim Porter
2022-09-05 21:48 ` Morgan Smith
2022-09-05 21:51 ` Lars Ingebrigtsen
[not found] ` <DM5PR03MB31639CBC75F62622AC4E70ABC57E9@DM5PR03MB3163.namprd03.prod.outlook.com>
2022-09-06 10:00 ` Lars Ingebrigtsen
2022-09-07 1:30 ` bug#57367: [PATCH V2] " Morgan Smith
2022-09-07 12:55 ` Lars Ingebrigtsen
2022-09-09 4:36 ` Jim Porter
2023-09-06 22:46 ` Stefan Kangas [this message]
2023-10-18 15:46 ` bug#57367: [PATCH v3] " Morgan Smith
2023-10-28 22:47 ` Jim Porter
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='CADwFkmnC8Y6kUHFDM5Lp3C2ZpieUejpAdMj_gFRY=q+p0TEAFA@mail.gmail.com' \
--to=stefankangas@gmail.com \
--cc=57367@debbugs.gnu.org \
--cc=Morgan.J.Smith@outlook.com \
--cc=jporterbugs@gmail.com \
--cc=larsi@gnus.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.