[Top][All Lists]
[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