Mercurial > hg > xemacs-beta
annotate lib-src/profile.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 | cd167465bf69 |
| children | b9167d522a9a |
| rev | line source |
|---|---|
| 428 | 1 /* profile.c --- generate periodic events for profiling of Emacs Lisp code. |
| 2 Copyright (C) 1992, 1994 Free Software Foundation, Inc. | |
| 2367 | 3 Copyright (C) 2004 Ben Wing. |
| 428 | 4 |
| 5 Author: Boaz Ben-Zvi <boaz@lcs.mit.edu> | |
| 6 | |
| 613 | 7 This file is part of XEmacs. |
| 428 | 8 |
| 613 | 9 XEmacs is free software; you can redistribute it and/or modify |
| 428 | 10 it under the terms of the GNU General Public License as published by |
| 11 the Free Software Foundation; either version 2, or (at your option) | |
| 12 any later version. | |
| 13 | |
| 613 | 14 XEmacs is distributed in the hope that it will be useful, |
| 428 | 15 but WITHOUT ANY WARRANTY; without even the implied warranty of |
| 16 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
| 17 GNU General Public License for more details. | |
| 18 | |
| 19 You should have received a copy of the GNU General Public License | |
| 613 | 20 along with XEmacs; see the file COPYING. If not, write to |
|
5287
cd167465bf69
More permission consistency.
Stephen J. Turnbull <stephen@xemacs.org>
parents:
2367
diff
changeset
|
21 the Free Software Foundation, Inc., 59 Temple Place - Suite 330, |
|
cd167465bf69
More permission consistency.
Stephen J. Turnbull <stephen@xemacs.org>
parents:
2367
diff
changeset
|
22 Boston, MA 02111-1307, USA. */ |
| 428 | 23 |
| 24 /* Synched up with: FSF 19.28. */ | |
| 25 /* #### Not sure if this is needed for XEmacs. */ | |
| 26 | |
| 27 /** | |
| 28 ** To be run as an emacs process. Input string that starts with: | |
| 29 ** 'z' -- resets the watch (to zero). | |
| 30 ** 'p' -- return time (on stdout) as string with format <sec>.<micro-sec> | |
| 31 ** 'q' -- exit. | |
| 32 ** | |
| 33 ** abstraction : a stopwatch | |
| 34 ** operations: reset_watch, get_time | |
| 35 */ | |
| 438 | 36 #include <config.h> |
| 428 | 37 #include <stdio.h> |
| 38 #include <stdlib.h> | |
| 39 #include "../src/systime.h" | |
| 40 | |
| 41 static struct timeval TV1, TV2; | |
| 42 static int watch_not_started = 1; /* flag */ | |
| 43 static char time_string[30]; | |
| 44 | |
| 442 | 45 #ifdef WIN32_NATIVE |
| 428 | 46 #include <sys/timeb.h> |
| 47 /* Emulate gettimeofday (Ulrich Leodolter, 1/11/95). */ | |
| 48 void | |
| 49 gettimeofday (struct timeval *tv, struct timezone *tz) | |
| 50 { | |
| 51 struct _timeb tb; | |
| 52 _ftime (&tb); | |
| 53 | |
| 54 tv->tv_sec = tb.time; | |
| 55 tv->tv_usec = tb.millitm * 1000L; | |
| 56 if (tz) | |
| 57 { | |
| 58 tz->tz_minuteswest = tb.timezone; /* minutes west of Greenwich */ | |
| 59 tz->tz_dsttime = tb.dstflag; /* type of dst correction */ | |
| 60 } | |
| 61 } | |
| 62 #endif | |
| 63 | |
| 64 /* Reset the stopwatch to zero. */ | |
| 65 | |
| 66 static void | |
| 67 reset_watch (void) | |
| 68 { | |
| 69 EMACS_GET_TIME (TV1); | |
| 70 watch_not_started = 0; | |
| 71 } | |
| 72 | |
| 73 /* This call returns the time since the last reset_watch call. The time | |
| 74 is returned as a string with the format <seconds>.<micro-seconds> | |
| 75 If reset_watch was not called yet, exit. */ | |
| 76 | |
| 77 static char * | |
| 78 get_time (void) | |
| 79 { | |
| 80 if (watch_not_started) | |
| 81 exit (1); /* call reset_watch first ! */ | |
| 82 EMACS_GET_TIME (TV2); | |
| 83 if (TV1.tv_usec > TV2.tv_usec) | |
| 84 { | |
| 85 TV2.tv_usec += 1000000; | |
| 86 TV2.tv_sec--; | |
| 87 } | |
| 88 sprintf (time_string, "%lu.%06lu", | |
| 89 (unsigned long) TV2.tv_sec - TV1.tv_sec, | |
| 90 (unsigned long) TV2.tv_usec - TV1.tv_usec); | |
| 91 return time_string; | |
| 92 } | |
| 93 | |
| 94 int | |
| 2367 | 95 main (int argc, char **argv) |
| 428 | 96 { |
| 97 int c; | |
| 98 while ((c = getchar ()) != EOF) | |
| 99 { | |
| 100 switch (c) | |
| 101 { | |
| 102 case 'z': | |
| 103 reset_watch (); | |
| 104 break; | |
| 105 case 'p': | |
| 106 puts (get_time ()); | |
| 107 break; | |
| 108 case 'q': | |
| 109 exit (0); | |
| 110 } | |
| 111 /* Anything remaining on the line is ignored. */ | |
| 112 while (c != '\n' && c != EOF) | |
| 113 c = getchar (); | |
| 114 } | |
| 115 return 1; | |
| 116 } |