diff options
| author | Paul Eggert | 2018-08-21 13:44:03 -0700 |
|---|---|---|
| committer | Paul Eggert | 2018-08-21 13:44:32 -0700 |
| commit | f18af6cd5cb7dbbf7420ec2d3efed4e202c4f0dd (patch) | |
| tree | 5f42e48e12a0ec77bd5cd5f32255a534635e89bf /lisp/term/common-win.el | |
| parent | 81e7eef8224c8a99a207b7a7b9dae1d598392ef7 (diff) | |
| download | emacs-f18af6cd5cb7dbbf7420ec2d3efed4e202c4f0dd.tar.gz emacs-f18af6cd5cb7dbbf7420ec2d3efed4e202c4f0dd.zip | |
Audit use of lsh and fix glitches
I audited use of lsh in the Lisp source code, and fixed the
glitches that I found. While I was at it, I replaced uses of lsh
with ash when either will do. Replacement is OK when either
argument is known to be nonnegative, or when only the low-order
bits of the result matter, and is a (minor) win since ash is a bit
more solid than lsh nowadays, and is a bit faster.
* lisp/calc/calc-ext.el (math-check-fixnum):
Prefer most-positive-fixnum to (lsh -1 -1).
* lisp/vc/vc-hg.el (vc-hg-state-fast): When testing fixnum width,
prefer (zerop (ash most-positive-fixnum -32)) to (zerop (lsh -1
32)) (Bug#32485#11).
* lisp/emacs-lisp/bytecomp.el (byte-compile-lapcode):
Tighten sanity-check for bytecode overflow, by checking that the
result of (ash pc -8) is nonnegative. Formerly this check was not
needed since lsh was used and the number overflowed differently.
* lisp/net/dns.el (dns-write): Fix some obvious sign typos in
shift counts. Evidently this part of the code has never been
exercised.
* lisp/progmodes/hideif.el (hif-shiftleft, hif-shiftright):
* lisp/term/common-win.el (x-setup-function-keys):
Simplify.
* admin/unidata/unidata-gen.el, admin/unidata/uvs.el:
* doc/lispref/keymaps.texi, doc/lispref/syntax.texi:
* doc/misc/calc.texi, doc/misc/cl.texi, etc/NEWS.19:
* lisp/arc-mode.el, lisp/calc/calc-bin.el, lisp/calc/calc-comb.el:
* lisp/calc/calc-ext.el, lisp/calc/calc-math.el:
* lisp/cedet/semantic/wisent/comp.el, lisp/composite.el:
* lisp/disp-table.el, lisp/dos-fns.el, lisp/edmacro.el:
* lisp/emacs-lisp/bindat.el, lisp/emacs-lisp/byte-opt.el:
* lisp/emacs-lisp/bytecomp.el, lisp/emacs-lisp/cl-extra.el:
* lisp/erc/erc-dcc.el, lisp/facemenu.el, lisp/gnus/message.el:
* lisp/gnus/nndoc.el, lisp/gnus/nnmaildir.el, lisp/image.el:
* lisp/international/ccl.el, lisp/international/fontset.el:
* lisp/international/mule-cmds.el, lisp/international/mule.el:
* lisp/json.el, lisp/mail/binhex.el, lisp/mail/rmail.el:
* lisp/mail/uudecode.el, lisp/md4.el, lisp/net/dns.el:
* lisp/net/ntlm.el, lisp/net/sasl.el, lisp/net/socks.el:
* lisp/net/tramp.el, lisp/obsolete/levents.el:
* lisp/obsolete/pgg-parse.el, lisp/org/org.el:
* lisp/org/ox-publish.el, lisp/progmodes/cc-defs.el:
* lisp/progmodes/ebnf2ps.el, lisp/progmodes/hideif.el:
* lisp/ps-bdf.el, lisp/ps-print.el, lisp/simple.el:
* lisp/tar-mode.el, lisp/term/common-win.el:
* lisp/term/tty-colors.el, lisp/term/xterm.el, lisp/vc/vc-git.el:
* lisp/vc/vc-hg.el, lisp/x-dnd.el, test/src/data-tests.el:
Prefer ash to lsh when either will do.
Diffstat (limited to 'lisp/term/common-win.el')
| -rw-r--r-- | lisp/term/common-win.el | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/lisp/term/common-win.el b/lisp/term/common-win.el index 6ef686a996f..a482067ef39 100644 --- a/lisp/term/common-win.el +++ b/lisp/term/common-win.el | |||
| @@ -59,20 +59,20 @@ | |||
| 59 | (setq system-key-alist | 59 | (setq system-key-alist |
| 60 | (list | 60 | (list |
| 61 | ;; These are special "keys" used to pass events from C to lisp. | 61 | ;; These are special "keys" used to pass events from C to lisp. |
| 62 | (cons (logior (lsh 0 16) 1) 'ns-power-off) | 62 | (cons 1 'ns-power-off) |
| 63 | (cons (logior (lsh 0 16) 2) 'ns-open-file) | 63 | (cons 2 'ns-open-file) |
| 64 | (cons (logior (lsh 0 16) 3) 'ns-open-temp-file) | 64 | (cons 3 'ns-open-temp-file) |
| 65 | (cons (logior (lsh 0 16) 4) 'ns-drag-file) | 65 | (cons 4 'ns-drag-file) |
| 66 | (cons (logior (lsh 0 16) 5) 'ns-drag-color) | 66 | (cons 5 'ns-drag-color) |
| 67 | (cons (logior (lsh 0 16) 6) 'ns-drag-text) | 67 | (cons 6 'ns-drag-text) |
| 68 | (cons (logior (lsh 0 16) 7) 'ns-change-font) | 68 | (cons 7 'ns-change-font) |
| 69 | (cons (logior (lsh 0 16) 8) 'ns-open-file-line) | 69 | (cons 8 'ns-open-file-line) |
| 70 | ;;; (cons (logior (lsh 0 16) 9) 'ns-insert-working-text) | 70 | ;;; (cons 9 'ns-insert-working-text) |
| 71 | ;;; (cons (logior (lsh 0 16) 10) 'ns-delete-working-text) | 71 | ;;; (cons 10 'ns-delete-working-text) |
| 72 | (cons (logior (lsh 0 16) 11) 'ns-spi-service-call) | 72 | (cons 11 'ns-spi-service-call) |
| 73 | (cons (logior (lsh 0 16) 12) 'ns-new-frame) | 73 | (cons 12 'ns-new-frame) |
| 74 | (cons (logior (lsh 0 16) 13) 'ns-toggle-toolbar) | 74 | (cons 13 'ns-toggle-toolbar) |
| 75 | (cons (logior (lsh 0 16) 14) 'ns-show-prefs) | 75 | (cons 14 'ns-show-prefs) |
| 76 | )))) | 76 | )))) |
| 77 | (set-terminal-parameter frame 'x-setup-function-keys t))) | 77 | (set-terminal-parameter frame 'x-setup-function-keys t))) |
| 78 | 78 | ||