emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Redundant if clause in xdisp.c?


From: Stefan Monnier
Subject: Re: Redundant if clause in xdisp.c?
Date: Sat, 29 Aug 2009 11:50:39 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1.50 (gnu/linux)

> Okay to remove the condition and to collapse both branches into a
> single code fragment?

Looks like a good idea, yes.  BTW, before playing with the redisplay,
please just throw away keyboard.c's handling of self-insert-command,
forward-char and backward-char, as well as the corresponding
"direct-rendering fast-path" code.

These optimizations seem to be pointless nowadays: even when running
Emacs-23 on my cell-phone (FreeRunner) displaying on my desktop, they
don't make any noticeable difference.

I refrained from installing this change for Emacs-23.2, out of fear
bigfoot might come and bite me, but I'll definitely install it for
Emacs-24.


        Stefan




reply via email to

[Prev in Thread] Current Thread [Next in Thread]