From: Dmitry Antipov <dmantipov@yandex.ru>
To: Stefan Monnier <monnier@IRO.UMontreal.CA>
Cc: emacs-devel@gnu.org
Subject: Re: [Emacs-diffs] /srv/bzr/emacs/trunk r109157: Compact buffers when idle.
Date: Thu, 19 Jul 2012 18:34:49 +0400 [thread overview]
Message-ID: <50081B09.5060406@yandex.ru> (raw)
In-Reply-To: <jwvsjcormxz.fsf-monnier+emacs@gnu.org>
On 07/19/2012 03:54 PM, Stefan Monnier wrote:
> I can't think of any case where that would provide any noticeable
> benefit (whereas I can think of cases where it just wastes resources).
Hm... I expect this to be something exotic, like replace-string
across hundreds of buffers. On the other side, it's quite
typical code refactoring operation. When I need to change function
name across the project, I'm using perl just because Emacs is too slow :-(.
> The GC is run fairly regularly, also typically while idle, and does the
> compaction.
1. Typically it happens when we're executing byte code, or from eval.c,
when we want to be as fast as possible.
2. I don't understand why idle call to Fgarbage_collect is glued with
auto-save-timeout. If I don't want to use auto-save, why I shouldn't
call GC when idle?
> So until you can provide some concrete and convincing numbers that show
> some benefit, I'd ask you to remove that pat of your patch.
Hm... I suppose that 3GHz CPUs makes a lot of small optimizations almost
invisible; but I believe it doesn't mean that they're not needed anymore.
For example, I'm pretty sure that removing Blength bytecode (and so calling
Flength via Funcall) will not introduce any noticeable slowdown for any
real use case (although we can design special benchmark which will
do myriads of (length xxx) and nothing more).
Dmitry
next prev parent reply other threads:[~2012-07-19 14:34 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <E1SrmrL-0004FM-0l@vcs.savannah.gnu.org>
2012-07-19 10:00 ` [Emacs-diffs] /srv/bzr/emacs/trunk r109157: Compact buffers when idle Stefan Monnier
2012-07-19 11:23 ` Dmitry Antipov
2012-07-19 11:54 ` Stefan Monnier
2012-07-19 14:34 ` Dmitry Antipov [this message]
2012-07-19 22:09 ` Stefan Monnier
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
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50081B09.5060406@yandex.ru \
--to=dmantipov@yandex.ru \
--cc=emacs-devel@gnu.org \
--cc=monnier@IRO.UMontreal.CA \
/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 public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).