Mercurial > hg > xemacs-beta
annotate src/vdb-win32.c @ 5366:f00192e1cd49
Examining the result of #'length: `eql', not `=', it's better style & cheaper
2011-03-08 Aidan Kehoe <kehoea@parhasard.net>
* buff-menu.el (list-buffers-noselect):
* byte-optimize.el (byte-optimize-identity):
* byte-optimize.el (byte-optimize-if):
* byte-optimize.el (byte-optimize-nth):
* byte-optimize.el (byte-optimize-nthcdr):
* bytecomp.el (byte-compile-warn-wrong-args):
* bytecomp.el (byte-compile-two-args-19->20):
* bytecomp.el (byte-compile-list):
* bytecomp.el (byte-compile-beginning-of-line):
* bytecomp.el (byte-compile-set):
* bytecomp.el (byte-compile-set-default):
* bytecomp.el (byte-compile-values):
* bytecomp.el (byte-compile-values-list):
* bytecomp.el (byte-compile-integerp):
* bytecomp.el (byte-compile-multiple-value-list-internal):
* bytecomp.el (byte-compile-throw):
* cl-macs.el (cl-do-arglist):
* cl-macs.el (cl-parse-loop-clause):
* cl-macs.el (multiple-value-bind):
* cl-macs.el (multiple-value-setq):
* cl-macs.el (get-setf-method):
* cmdloop.el (command-error):
* cmdloop.el (y-or-n-p-minibuf):
* cmdloop.el (yes-or-no-p-minibuf):
* coding.el (unencodable-char-position):
* cus-edit.el (custom-face-prompt):
* cus-edit.el (custom-buffer-create-internal):
* cus-edit.el (widget-face-action):
* cus-edit.el (custom-group-value-create):
* descr-text.el (describe-char-unicode-data):
* dialog-gtk.el (popup-builtin-question-dialog):
* dragdrop.el (experimental-dragdrop-drop-log-function):
* dragdrop.el (experimental-dragdrop-drop-mime-default):
* easymenu.el (easy-menu-add):
* easymenu.el (easy-menu-remove):
* faces.el (read-face-name):
* faces.el (set-face-stipple):
* files.el (file-name-non-special):
* font.el (font-combine-fonts):
* font.el (font-set-face-font):
* font.el (font-parse-rgb-components):
* font.el (font-rgb-color-p):
* font.el (font-color-rgb-components):
* gnuserv.el (gnuserv-edit-files):
* help.el (key-or-menu-binding):
* help.el (function-documentation-1):
* help.el (function-documentation):
* info.el (info):
* isearch-mode.el (isearch-exit):
* isearch-mode.el (isearch-edit-string):
* isearch-mode.el (isearch-*-char):
* isearch-mode.el (isearch-complete1):
* ldap.el (ldap-encode-country-string):
* ldap.el (ldap-decode-string):
* minibuf.el (read-file-name-internal-1):
* minibuf.el (read-non-nil-coding-system):
* minibuf.el (get-user-response):
* mouse.el (drag-window-divider):
* mule/ccl.el:
* mule/ccl.el (ccl-compile-if):
* mule/ccl.el (ccl-compile-break):
* mule/ccl.el (ccl-compile-repeat):
* mule/ccl.el (ccl-compile-write-repeat):
* mule/ccl.el (ccl-compile-call):
* mule/ccl.el (ccl-compile-end):
* mule/ccl.el (ccl-compile-read-multibyte-character):
* mule/ccl.el (ccl-compile-write-multibyte-character):
* mule/ccl.el (ccl-compile-translate-character):
* mule/ccl.el (ccl-compile-mule-to-unicode):
* mule/ccl.el (ccl-compile-unicode-to-mule):
* mule/ccl.el (ccl-compile-lookup-integer):
* mule/ccl.el (ccl-compile-lookup-character):
* mule/ccl.el (ccl-compile-map-multiple):
* mule/ccl.el (ccl-compile-map-single):
* mule/devan-util.el (devanagari-compose-to-one-glyph):
* mule/devan-util.el (devanagari-composition-component):
* mule/mule-cmds.el (finish-set-language-environment):
* mule/viet-util.el:
* mule/viet-util.el (viet-encode-viscii-char):
* multicast.el (open-multicast-group):
* newcomment.el (comment-quote-nested):
* newcomment.el (comment-region):
* newcomment.el (comment-dwim):
* regexp-opt.el (regexp-opt-group):
* replace.el (map-query-replace-regexp):
* specifier.el (derive-device-type-from-tag-set):
* subr.el (skip-chars-quote):
* test-harness.el (test-harness-from-buffer):
* test-harness.el (batch-test-emacs):
* wid-edit.el (widget-choice-action):
* wid-edit.el (widget-symbol-prompt-internal):
* wid-edit.el (widget-color-action):
* window-xemacs.el (push-window-configuration):
* window-xemacs.el (pop-window-configuration):
* window.el (quit-window):
* x-compose.el (electric-diacritic):
It's better style, and cheaper (often one assembler instruction
vs. a C funcall in the byte code), to use `eql' instead of `='
when it's clear what numerical type a given result will be. Change
much of our code to do this, with the help of a byte-compiler
change (not comitted) that looked for calls to #'length (which
always returns an integer) in its args.
| author | Aidan Kehoe <kehoea@parhasard.net> |
|---|---|
| date | Tue, 08 Mar 2011 23:41:52 +0000 |
| parents | f395ee7ad844 |
| children | 308d34e9f07d |
| rev | line source |
|---|---|
| 3092 | 1 /* Virtual diry bit implementation for XEmacs. |
| 2 Copyright (C) 2005 Marcus Crestani. | |
| 3 | |
| 4 This file is part of XEmacs. | |
| 5 | |
| 6 XEmacs is free software; you can redistribute it and/or modify it | |
| 7 under the terms of the GNU General Public License as published by the | |
| 8 Free Software Foundation; either version 2, or (at your option) any | |
| 9 later version. | |
| 10 | |
| 11 XEmacs is distributed in the hope that it will be useful, but WITHOUT | |
| 12 ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
| 13 FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
| 14 for more details. | |
| 15 | |
| 16 You should have received a copy of the GNU General Public License | |
| 17 along with XEmacs; see the file COPYING. If not, write to | |
| 18 the Free Software Foundation, Inc., 59 Temple Place - Suite 330, | |
| 19 Boston, MA 02111-1307, USA. */ | |
| 20 | |
| 21 /* Synched up with: Not in FSF. */ | |
| 22 | |
| 23 #include <config.h> | |
| 24 #include "lisp.h" | |
| 25 #include "gc.h" | |
| 26 #include "mc-alloc.h" | |
| 27 #include "vdb.h" | |
| 28 | |
| 29 #include "syswindows.h" | |
| 30 | |
| 31 | |
|
5042
f395ee7ad844
Fix some compile warnings, make vdb test code conditional on DEBUG_XEMACS
Ben Wing <ben@xemacs.org>
parents:
4827
diff
changeset
|
32 static LONG WINAPI |
| 3092 | 33 win32_fault_handler (LPEXCEPTION_POINTERS e) |
| 34 { | |
| 35 #define GET_FAULT_ADDRESS (void *) e->ExceptionRecord->ExceptionInformation[1] | |
| 36 if ((e->ExceptionRecord->ExceptionCode == EXCEPTION_ACCESS_VIOLATION) | |
| 37 && (e->ExceptionRecord->ExceptionInformation[0] == 1) | |
| 38 && write_barrier_enabled | |
| 39 && (fault_on_protected_page (GET_FAULT_ADDRESS))) | |
| 40 { | |
| 41 vdb_designate_modified (GET_FAULT_ADDRESS); | |
| 42 unprotect_page_and_mark_dirty (GET_FAULT_ADDRESS); | |
| 43 return EXCEPTION_CONTINUE_EXECUTION; | |
| 44 } | |
| 45 else | |
| 46 return EXCEPTION_CONTINUE_SEARCH; | |
| 47 } | |
| 48 | |
|
4827
11daf37dae4d
more fixes to get a clean compile
Ben Wing <ben@xemacs.org>
parents:
3092
diff
changeset
|
49 typedef LONG (WINAPI *gcPVECTORED_EXCEPTION_HANDLER) (LPEXCEPTION_POINTERS e); |
| 3092 | 50 |
| 51 | |
| 52 void | |
| 53 vdb_install_signal_handler (void) | |
| 54 { | |
| 55 HMODULE hm; | |
| 56 PVOID (WINAPI *aveh) (ULONG, gcPVECTORED_EXCEPTION_HANDLER); | |
| 57 | |
| 58 /* See init_signals_very_early () in signal.c. */ | |
| 59 if (noninteractive && !initialized) | |
| 60 { | |
| 61 allow_incremental_gc = 0; | |
| 62 return; | |
| 63 } | |
| 64 | |
| 65 hm = qxeGetModuleHandle (XETEXT ("kernel32")); | |
| 66 if (hm) | |
| 67 aveh = (PVOID (WINAPI *) (ULONG, gcPVECTORED_EXCEPTION_HANDLER)) | |
| 68 GetProcAddress (hm, "AddVectoredExceptionHandler"); | |
| 69 else | |
| 70 { | |
| 71 fprintf (stderr, "\nFAILED TO LOAD LIBRARY\n"); | |
| 72 aveh = NULL; | |
| 73 } | |
| 74 if (aveh) | |
| 75 { | |
| 76 allow_incremental_gc = 1; | |
| 77 aveh (TRUE, win32_fault_handler); | |
| 78 } | |
| 79 else | |
| 80 { | |
| 81 fprintf (stderr, "\nFAILED TO INSTALL SIGNAL HANDLER\n"); | |
| 82 ABORT (); | |
| 83 } | |
| 84 } | |
| 85 | |
| 86 | |
| 87 void | |
| 88 vdb_protect (void *ptr, EMACS_INT len) | |
| 89 { | |
| 90 DWORD old; | |
| 91 VirtualProtect (ptr, len, PAGE_READONLY, &old); | |
| 92 } | |
| 93 | |
| 94 | |
| 95 void | |
| 96 vdb_unprotect (void *ptr, EMACS_INT len) | |
| 97 { | |
| 98 DWORD old; | |
| 99 VirtualProtect (ptr, len, PAGE_READWRITE, &old); | |
| 100 } |