]> code.delx.au - gnu-emacs-elpa/blobdiff - benchmark/context-coloring-benchmark.el
Refactor benchmarks. Improve benchmark reliability.
[gnu-emacs-elpa] / benchmark / context-coloring-benchmark.el
index 887cdc830d9c56e619091bcaf6830461fc8f06d1..19fa4a366d3a53ea9bd4eae95b776951ba8b58f0 100644 (file)
@@ -1,4 +1,4 @@
-;;; benchmark/context-coloring-benchmark.el --- Benchmarks for context coloring. -*- lexical-binding: t; -*-
+;;; context-coloring-benchmark.el --- Benchmarks for context coloring  -*- lexical-binding: t; -*-
 
 ;; Copyright (C) 2014-2015  Free Software Foundation, Inc.
 
 ;; You should have received a copy of the GNU General Public License
 ;; along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
+;;; Commentary:
+
+;; Benchmarks for context coloring.
+
+;; Use with `make bench'.
+
 ;;; Code:
 
+(require 'context-coloring)
+(require 'js2-mode)
+
+
 (defconst context-coloring-benchmark-path
   (file-name-directory (or load-file-name buffer-file-name))
   "This file's directory.")
   "Resolve PATH from this file's directory."
   (expand-file-name path context-coloring-benchmark-path))
 
+(defun context-coloring-benchmark-series (sequence callback)
+  "Call each function in SEQUENCE, then call CALLBACK.  Each
+function is passed a single callback parameter for it to call
+when it is done."
+  (cond
+   ((null sequence)
+    (funcall callback))
+   (t
+    (funcall
+     (car sequence)
+     (lambda ()
+       (run-with-timer
+        nil nil
+        (lambda ()
+          (context-coloring-benchmark-series
+           (cdr sequence)
+           callback))))))))
+
+(defun context-coloring-benchmark-mapc (sequence iteratee callback)
+  "For each element in SEQUENCE, call ITERATEE, finally call
+CALLBACK.  ITERATEE is passed the current element and a callback
+for it to call when it is done."
+  (cond
+   ((null sequence)
+    (funcall callback))
+   (t
+    (funcall
+     iteratee
+     (car sequence)
+     (lambda ()
+       (run-with-timer
+        nil nil
+        (lambda ()
+          (context-coloring-benchmark-mapc
+           (cdr sequence)
+           iteratee
+           callback))))))))
+
 (defun context-coloring-benchmark-log-results (result-file fixture)
-  "Log benchmarking results for FIXTURE to RESULT-FILE."
+  "Log benchmarking results to RESULT-FILE for fixture FIXTURE."
   (elp-results)
   (let ((results-buffer (current-buffer)))
     (with-temp-buffer
   (make-directory (context-coloring-benchmark-resolve-path "./logs") t)
   (append-to-file nil nil result-file))
 
-(defun context-coloring-benchmark-next-tick (function)
-  "Defer execution of FUNCTION to clear the stack and to ensure
-asynchrony."
-  (run-at-time 0.001 nil function))
-
-(defun context-coloring-benchmark-next (list continue stop)
-  "Run the next test in LIST by calling CONTINUE.  When LIST is
-exhausted, call STOP instead."
-  (if (null list)
-      (progn
-        (context-coloring-benchmark-next-tick stop))
-    (context-coloring-benchmark-next-tick
-     (lambda ()
-       (funcall
-        continue
-        (car list)
-        (lambda ()
-          (context-coloring-benchmark-next (cdr list) continue stop)))))))
-
-(defun context-coloring-benchmark-async (title setup teardown fixtures callback)
-  "Measure the performance of all FIXTURES, calling CALLBACK when
-all are done."
+(defun context-coloring-benchmark (title setup teardown fixtures callback)
+  "Execute a benchmark titled TITLE with SETUP and TEARDOWN
+callbacks.  Measure the performance of all FIXTURES, calling
+CALLBACK when all are done."
   (funcall setup)
+  (elp-instrument-package "context-coloring-")
   (let ((result-file (context-coloring-benchmark-resolve-path
                       (format "./logs/results-%s-%s.log"
                               title (format-time-string "%s")))))
-    (context-coloring-benchmark-next
+    (context-coloring-benchmark-mapc
      fixtures
-     (lambda (path next)
+     (lambda (path callback)
        (let ((fixture (context-coloring-benchmark-resolve-path path))
              advice)
          (setq
@@ -80,20 +111,21 @@ all are done."
                (lambda ()
                  (setq count (+ count 1))
                  ;; Test 5 times.
-                 (if (= count 5)
-                     (progn
-                       (advice-remove 'context-coloring-colorize advice)
-                       (kill-buffer)
-                       (context-coloring-benchmark-log-results
-                        result-file
-                        fixture)
-                       (funcall next))
-                   (funcall 'context-coloring-colorize)))))))
+                 (cond
+                  ((= count 5)
+                   (advice-remove 'context-coloring-colorize advice)
+                   (kill-buffer)
+                   (context-coloring-benchmark-log-results
+                    result-file
+                    fixture)
+                   (funcall callback))
+                  (t
+                   (funcall 'context-coloring-colorize))))))))
          (advice-add 'context-coloring-colorize :around advice)
          (find-file fixture)))
      (lambda ()
        (funcall teardown)
-       (when callback (funcall callback))))))
+       (funcall callback)))))
 
 (defconst context-coloring-benchmark-js-fixtures
   '("./fixtures/jquery-2.1.1.js"
@@ -102,57 +134,44 @@ all are done."
     "./fixtures/mkdirp-0.5.0.js")
   "Arbitrary JavaScript files for performance scrutiny.")
 
-(defun context-coloring-benchmark-js-mode-setup ()
-  "Preparation logic for `js-mode'."
-  (add-hook 'js-mode-hook 'context-coloring-mode)
-  (elp-instrument-package "context-coloring-"))
-
-(defun context-coloring-benchmark-js-mode-teardown ()
-  "Cleanup logic for `js-mode'."
-  (remove-hook 'js-mode-hook 'context-coloring-mode))
-
 (defun context-coloring-benchmark-js-mode-run (callback)
   "Benchmark `js-mode', then call CALLBACK."
-  (context-coloring-benchmark-async
+  (context-coloring-benchmark
    "js-mode"
-   'context-coloring-benchmark-js-mode-setup
-   'context-coloring-benchmark-js-mode-teardown
+   (lambda ()
+     "Preparation logic for `js-mode'."
+     (add-hook 'js-mode-hook 'context-coloring-mode))
+   (lambda ()
+     "Cleanup logic for `js-mode'."
+     (remove-hook 'js-mode-hook 'context-coloring-mode))
    context-coloring-benchmark-js-fixtures
    callback))
 
-(defun context-coloring-benchmark-js2-mode-setup ()
-  "Preparation logic for `js2-mode'."
-  (require 'js2-mode)
-  (setq js2-mode-show-parse-errors nil)
-  (setq js2-mode-show-strict-warnings nil)
-  (add-to-list 'auto-mode-alist '("\\.js\\'" . js2-mode))
-  (add-hook 'js2-mode-hook 'context-coloring-mode)
-  (elp-instrument-package "context-coloring-"))
-
-(defun context-coloring-benchmark-js2-mode-teardown ()
-  "Cleanup logic for `js2-mode'."
-  (remove-hook 'js2-mode-hook 'context-coloring-mode)
-  (setq auto-mode-alist (delete '("\\.js\\'" . js2-mode)
-                                auto-mode-alist))
-  (setq js2-mode-show-strict-warnings t)
-  (setq js2-mode-show-parse-errors t))
-
 (defun context-coloring-benchmark-js2-mode-run (callback)
   "Benchmark `js2-mode', then call CALLBACK."
-  (context-coloring-benchmark-async
+  (context-coloring-benchmark
    "js2-mode"
-   'context-coloring-benchmark-js2-mode-setup
-   'context-coloring-benchmark-js2-mode-teardown
+   (lambda ()
+     "Preparation logic for `js2-mode'."
+     (setq js2-mode-show-parse-errors nil)
+     (setq js2-mode-show-strict-warnings nil)
+     (add-to-list 'auto-mode-alist '("\\.js\\'" . js2-mode))
+     (add-hook 'js2-mode-hook 'context-coloring-mode))
+   (lambda ()
+     "Cleanup logic for `js2-mode'."
+     (remove-hook 'js2-mode-hook 'context-coloring-mode)
+     (setq auto-mode-alist (delete '("\\.js\\'" . js2-mode)
+                                   auto-mode-alist))
+     (setq js2-mode-show-strict-warnings t)
+     (setq js2-mode-show-parse-errors t))
    context-coloring-benchmark-js-fixtures
    callback))
 
 (defun context-coloring-benchmark-run ()
   "Benchmark all modes, then exit."
-  (context-coloring-benchmark-next
+  (context-coloring-benchmark-series
    '(context-coloring-benchmark-js-mode-run
      context-coloring-benchmark-js2-mode-run)
-   (lambda (function next)
-     (funcall function next))
    (lambda ()
      (kill-emacs))))