all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
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




  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

* 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 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.