]> asedeno.scripts.mit.edu Git - cl-protobufs.git/blobdiff - wire-format.lisp
Don't kluge *asdf-verbose* on asdf3.
[cl-protobufs.git] / wire-format.lisp
index e84dc4f897541202965fe35d506dcc51c6769055..6d597decced6dc4b9fef97d9c23411ee66ca5ae6 100644 (file)
@@ -1,8 +1,8 @@
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 ;;;                                                                  ;;;
-;;; Confidential and proprietary information of ITA Software, Inc.   ;;;
+;;; Free Software published under an MIT-like license. See LICENSE   ;;;
 ;;;                                                                  ;;;
-;;; Copyright (c) 2012 ITA Software, Inc.  All rights reserved.      ;;;
+;;; Copyright (c) 2012 Google, Inc.  All rights reserved.            ;;;
 ;;;                                                                  ;;;
 ;;; Original author: Scott McKay                                     ;;;
 ;;;                                                                  ;;;
 
 (eval-when (:compile-toplevel :load-toplevel :execute)
 
+;; If you need to debug the (de)serializer, (pushnew :debug-serialization *features*)
+;; Otherwise, we try to make (de)serialization as fast as possible,
+;; risking life and limb to do so
+(defparameter $optimize-serialization
+  #+debug-serialization $optimize-default
+  #-debug-serialization $optimize-fast-unsafe)
+
 (defconstant $wire-type-varint 0)
 (defconstant $wire-type-64bit  1)
 (defconstant $wire-type-string 2)
-(defconstant $wire-type-start-group 3)          ;supposedly obsolete
-(defconstant $wire-type-end-group   4)          ;supposedly obsolete
+(defconstant $wire-type-start-group 3)          ;supposedly deprecated, but no such luck
+(defconstant $wire-type-end-group   4)          ;supposedly deprecated
 (defconstant $wire-type-32bit  5)
 
 )       ;eval-when
@@ -30,7 +37,7 @@
 (defun make-tag (type index)
   "Given a wire type or the name of a Protobufs type and a field index,
    return the tag that encodes both of them."
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (locally (declare #.$optimize-serialization)
     (if (typep type 'fixnum)
       (ilogior type (iash index 3))
       (let ((type (ecase type
@@ -49,6 +56,7 @@
         (ilogior type (iash index 3))))))
 
 (define-compiler-macro make-tag (&whole form type index)
+  (setq type (fold-symbol type))
   (cond ((typep type 'fixnum)
          `(ilogior ,type (iash ,index 3)))
         ((keywordp type)
            `(ilogior ,type (iash ,index 3))))
         (t form)))
 
-
-(defun zig-zag-encode32 (val)
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (signed-byte 32) val))
-  (logxor (ash val 1) (ash val -31)))
-
-(defun zig-zag-encode64 (val)
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (signed-byte 64) val))
-  (logxor (ash val 1) (ash val -63)))
-
-(define-compiler-macro zig-zag-encode32 (&whole form val)
-  (if (atom val)
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0))
-                       (type (signed-byte 32) ,val))
-       (logxor (ash ,val 1) (ash ,val -31)))
-    form))
-
-(define-compiler-macro zig-zag-encode64 (&whole form val)
-  (if (atom val)
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0))
-                       (type (signed-byte 64) ,val))
-       (logxor (ash ,val 1) (ash ,val -63)))
-    form))
-
-(defun zig-zag-decode32 (val)
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (logxor (ash val -1) (- (logand val 1))))
-
-(defun zig-zag-decode64 (val)
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (logxor (ash val -1) (- (logand val 1))))
-
-(define-compiler-macro zig-zag-decode32 (&whole form val)
-  (if (atom val)
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0)))
-       (logxor (ash ,val -1) (- (logand ,val 1))))
-    form))
-
-(define-compiler-macro zig-zag-decode64 (&whole form val)
-  (if (atom val)
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0)))
-       (logxor (ash ,val -1) (- (logand ,val 1))))
-    form))
+(defun fold-symbol (x)
+  "Given an expression 'x', constant-fold it until it can be folded no more."
+  (let ((last '#:last))
+    (loop
+      (cond ((eq x last) (return x))
+            ((and (listp x)
+                  (eq (first x) 'quote)
+                  (constantp (second x)))
+             (shiftf last x (second x)))
+            ((and (symbolp x)
+                  (boundp x))
+             (shiftf last x (symbol-value x)))
+            (t (return x))))))
+
+
+(defmacro gen-zig-zag (bits)
+  "Generate 32- or 64-bit versions of zig-zag encoder/decoder."
+  (assert (and (plusp bits) (zerop (mod bits 8))))
+  (let* ((zig-zag-encode (fintern "~A~A" 'zig-zag-encode bits))
+         (zig-zag-decode (fintern "~A~A" 'zig-zag-decode bits))
+         (zig-zag-shift (1+ (- bits))))
+    `(progn
+       (defun ,zig-zag-encode (val)
+         (declare #.$optimize-serialization)
+         (declare (type (signed-byte ,bits) val))
+         (logxor (ash val 1) (ash val ,zig-zag-shift)))
+       (define-compiler-macro ,zig-zag-encode (&whole form val)
+         (if (atom val)
+           `(locally (declare #.$optimize-serialization
+                              (type (signed-byte ,',bits) ,val))
+              (logxor (ash ,val 1) (ash ,val ,',zig-zag-shift)))
+           form))
+       (defun ,zig-zag-decode (val)
+         (declare #.$optimize-serialization)
+         (declare (type (unsigned-byte ,bits) val))
+         (logxor (ash val -1) (- (logand val 1))))
+       (define-compiler-macro ,zig-zag-decode (&whole form val)
+         (if (atom val)
+           `(locally (declare #.$optimize-serialization
+                              (type (unsigned-byte ,',bits) ,val))
+              (logxor (ash ,val -1) (- (logand ,val 1))))
+           form)))))
+
+(gen-zig-zag 32)
+(gen-zig-zag 64)
 
 
 ;;; Serializers
    The value is given by 'val', the primitive type by 'type'.
    Modifies the buffer in place, and returns the new index into the buffer.
    Watch out, this function turns off most type checking and all array bounds checking."
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
-           (type (unsigned-byte 32) tag)
+  (declare (type (unsigned-byte 32) tag)
            (type fixnum index))
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (locally (declare #.$optimize-serialization)
     (let ((idx (encode-uint32 tag buffer index)))
       (declare (type fixnum idx))
       (ecase type
-        ((:int32 :uint32)
+        ((:int32 :int64)
+         (encode-int val buffer idx))
+        ((:uint32)
          (encode-uint32 val buffer idx))
-        ((:int64 :uint64)
+        ((:uint64)
          (encode-uint64 val buffer idx))
         ((:sint32)
          (encode-uint32 (zig-zag-encode32 val) buffer idx))
          (encode-uint64 val buffer idx))))))
 
 (define-compiler-macro serialize-prim (&whole form val type tag buffer index)
+  (setq type (fold-symbol type)
+        tag  (fold-symbol tag))
   (if (member type '(:int32 :uint32 :int64 :uint64 :sint32 :sint64
                      :fixed32 :sfixed32 :fixed64 :sfixed64
                      :string :bytes :bool :float :double))
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0))
+    `(locally (declare #.$optimize-serialization
                        (type (simple-array (unsigned-byte 8)) ,buffer)
                        ;; 'tag' is a constant, no need to declare its type
                        (type fixnum ,index))
        (let ((idx (encode-uint32 ,tag ,buffer ,index)))
          (declare (type fixnum idx))
          ,(ecase type
-            ((:int32 :uint32)
+            ((:int32 :int64)
+             `(encode-int ,val ,buffer idx))
+            ((:uint32)
              `(encode-uint32 ,val ,buffer idx))
-            ((:int64 :uint64)
+            ((:uint64)
              `(encode-uint64 ,val ,buffer idx))
             ((:sint32)
              `(encode-uint32 (zig-zag-encode32 ,val) ,buffer idx))
              `(encode-double ,val ,buffer idx)))))
     form))
 
-(defun serialize-packed (values type tag buffer index)
+(defun serialize-packed (values type tag buffer index &optional vectorp)
   "Serializes a set of packed values into the buffer at the given index.
    The values are given by 'values', the primitive type by 'type'.
    Modifies the buffer in place, and returns the new index into the buffer.
    Watch out, this function turns off most type checking and all array bounds checking."
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
+  (declare (ignore vectorp)
+           (type (simple-array (unsigned-byte 8)) buffer)
            (type (unsigned-byte 32) tag)
            (type fixnum index))
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (locally (declare #.$optimize-serialization)
     (let ((idx (encode-uint32 tag buffer index)))
       (declare (type fixnum idx))
       (multiple-value-bind (full-len len)
           (packed-size values type tag)
         (declare (type fixnum len) (ignore full-len))
         (setq idx (encode-uint32 len buffer idx)))
-      (ecase type
-        ((:int32 :uint32)
-         (dolist (val values idx)
-           (setq idx (encode-uint32 val buffer idx))))
-        ((:int64 :uint64)
-         (dolist (val values idx)
-           (setq idx (encode-uint64 val buffer idx))))
-        ((:sint32)
-         (dolist (val values idx)
-           (setq idx (encode-uint32 (zig-zag-encode32 val) buffer idx))))
-        ((:sint64)
-         (dolist (val values idx)
-           (setq idx (encode-uint64 (zig-zag-encode64 val) buffer idx))))
-        ((:fixed32)
-         (dolist (val values idx)
-           (setq idx (encode-fixed32 val buffer idx))))
-        ((:sfixed32)
-         (dolist (val values idx)
-           (setq idx (encode-sfixed32 val buffer idx))))
-        ((:fixed64)
-         (dolist (val values idx)
-           (setq idx (encode-fixed64 val buffer idx))))
-        ((:sfixed64)
-         (dolist (val values idx)
-           (setq idx (encode-sfixed64 val buffer idx))))
-        ((:float)
-         (dolist (val values idx)
-           (setq idx (encode-single val buffer idx))))
-        ((:double)
-         (dolist (val values idx)
-           (setq idx (encode-double val buffer idx))))))))
-
-(define-compiler-macro serialize-packed (&whole form values type tag buffer index)
-  (if (member type '(:int32 :uint32 :int64 :uint64 :sint32 :sint64
-                     :fixed32 :sfixed32 :fixed64 :sfixed64
-                     :float :double))
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0))
+      (map ()
+           (ecase type
+             ((:int32 :int64) #'(lambda (val) (setq idx (encode-int val buffer idx))))
+             ((:uint32) #'(lambda (val) (setq idx (encode-uint32 val buffer idx))))
+             ((:uint64) #'(lambda (val) (setq idx (encode-uint64 val buffer idx))))
+             ((:sint32) #'(lambda (val) (setq idx (encode-uint32 (zig-zag-encode32 val) buffer idx))))
+             ((:sint64) #'(lambda (val) (setq idx (encode-uint64 (zig-zag-encode64 val) buffer idx))))
+             ((:fixed32) #'(lambda (val) (setq idx (encode-fixed32 val buffer idx))))
+             ((:sfixed32) #'(lambda (val) (setq idx (encode-sfixed32 val buffer idx))))
+             ((:fixed64) #'(lambda (val) (setq idx (encode-fixed64 val buffer idx))))
+             ((:sfixed64) #'(lambda (val) (setq idx (encode-sfixed64 val buffer idx))))
+             ((:bool) #'(lambda (val) (setq idx (encode-uint32 (if val 1 0) buffer idx))))
+             ((:float) #'(lambda (val) (setq idx (encode-single val buffer idx))))
+             ((:double) #'(lambda (val) (setq idx (encode-double val buffer idx)))))
+           values)
+      idx)))
+
+;; The optimized serializers supply 'vectorp' so we can generate better code
+(define-compiler-macro serialize-packed (&whole form values type tag buffer index
+                                         &optional (vectorp nil vectorp-p))
+  (setq type (fold-symbol type)
+        tag  (fold-symbol tag))
+  (if (and vectorp-p
+           `(member type '(:int32 :uint32 :int64 :uint64 :sint32 :sint64
+                           :fixed32 :sfixed32 :fixed64 :sfixed64
+                           :bool :float :double)))
+    `(locally (declare #.$optimize-serialization
                        (type (simple-array (unsigned-byte 8)) ,buffer)
+                       ;; 'tag' is a constant, no need to declare its type
                        (type fixnum ,index))
        (let ((idx (encode-uint32 ,tag ,buffer ,index)))
          (declare (type fixnum idx))
              (packed-size ,values ,type ,tag)
            (declare (type fixnum len) (ignore full-len))
            (setq idx (encode-uint32 len ,buffer idx)))
-         (dolist (val ,values idx)
-           ,(ecase type
-              ((:int32 :uint32)
-               `(setq idx (encode-uint32 val ,buffer idx)))
-              ((:int64 :uint64)
-               `(setq idx (encode-uint64 val ,buffer idx)))
-              ((:sint32)
-               `(setq idx (encode-uint32 (zig-zag-encode32 val) ,buffer idx)))
-              ((:sint64)
-               `(setq idx (encode-uint64 (zig-zag-encode64 val) ,buffer idx)))
-              ((:fixed32)
-               `(setq idx (encode-fixed32 val ,buffer idx)))
-              ((:sfixed32)
-               `(setq idx (encode-sfixed32 val ,buffer idx)))
-              ((:fixed64)
-               `(setq idx (encode-fixed64 val ,buffer idx)))
-              ((:sfixed64)
-               `(setq idx (encode-sfixed64 val ,buffer idx)))
-              ((:float)
-               `(setq idx (encode-single val ,buffer idx)))
-              ((:double)
-               `(setq idx (encode-double val ,buffer idx)))))))
+         (,(if vectorp 'dovector 'dolist) (val ,values)
+            ,(ecase type
+               ((:int32 :int64)
+                `(setq idx (encode-int val ,buffer idx)))
+               ((:uint32)
+                `(setq idx (encode-uint32 val ,buffer idx)))
+               ((:uint64)
+                `(setq idx (encode-uint64 val ,buffer idx)))
+               ((:sint32)
+                `(setq idx (encode-uint32 (zig-zag-encode32 val) ,buffer idx)))
+               ((:sint64)
+                `(setq idx (encode-uint64 (zig-zag-encode64 val) ,buffer idx)))
+               ((:fixed32)
+                `(setq idx (encode-fixed32 val ,buffer idx)))
+               ((:sfixed32)
+                `(setq idx (encode-sfixed32 val ,buffer idx)))
+               ((:fixed64)
+                `(setq idx (encode-fixed64 val ,buffer idx)))
+               ((:sfixed64)
+                `(setq idx (encode-sfixed64 val ,buffer idx)))
+               ((:bool)
+                `(setq idx (encode-uint32 (if val 1 0) ,buffer idx)))
+               ((:float)
+                `(setq idx (encode-single val ,buffer idx)))
+               ((:double)
+                `(setq idx (encode-double val ,buffer idx)))))
+         idx))
     form))
 
-(defun serialize-enum (val values tag buffer index)
+(defun serialize-enum (val enum-values tag buffer index)
   "Serializes a Protobufs enum value into the buffer at the given index.
-   The value is given by 'val', the enum values are in 'values'.
+   The value is given by 'val', the enum values are in 'enum-values'.
    Modifies the buffer in place, and returns the new index into the buffer.
    Watch out, this function turns off most type checking and all array bounds checking."
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
+  (declare (type list enum-values)
+           (type (simple-array (unsigned-byte 8)) buffer)
            (type (unsigned-byte 32) tag)
            (type fixnum index))
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
-    (let* ((val (let ((e (find val values :key #'proto-value)))
+  (locally (declare #.$optimize-serialization)
+    (let* ((val (let ((e (find val enum-values :key #'proto-value)))
                   (and e (proto-index e))))
            (idx (encode-uint32 tag buffer index)))
       (declare (type (unsigned-byte 32) val)
                (type fixnum idx))
       (encode-uint32 val buffer idx))))
 
+(defun serialize-packed-enum (values enum-values tag buffer index)
+  "Serializes Protobufs enum values into the buffer at the given index.
+   The values are given by 'values', the enum values are in 'enum-values'.
+   Modifies the buffer in place, and returns the new index into the buffer.
+   Watch out, this function turns off most type checking and all array bounds checking."
+  (declare (type list enum-values)
+           (type (simple-array (unsigned-byte 8)) buffer)
+           (type (unsigned-byte 32) tag)
+           (type fixnum index))
+  (locally (declare #.$optimize-serialization)
+    (let ((idx (encode-uint32 tag buffer index)))
+      (declare (type fixnum idx))
+      (multiple-value-bind (full-len len)
+          (packed-enum-size values enum-values tag)
+        (declare (type fixnum len) (ignore full-len))
+        (setq idx (encode-uint32 len buffer idx)))
+      (map () #'(lambda (val)
+                  (let ((val (let ((e (find val enum-values :key #'proto-value)))
+                               (and e (proto-index e)))))
+                    (declare (type (unsigned-byte 32) val))
+                    (setq idx (encode-uint32 val buffer idx)))) values)
+      idx)))
+
 
 ;;; Deserializers
 
    Watch out, this function turns off most type checking and all array bounds checking."
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (locally (declare #.$optimize-serialization)
     (ecase type
-      ((:int32 :uint32)
+      ((:int32 :int64)
+       (decode-int buffer index))
+      ((:uint32)
        (decode-uint32 buffer index))
-      ((:int64 :uint64)
+      ((:uint64)
        (decode-uint64 buffer index))
       ((:sint32)
        (multiple-value-bind (val idx)
        (decode-uint64 buffer index)))))
 
 (define-compiler-macro deserialize-prim (&whole form type buffer index)
+  (setq type (fold-symbol type))
   (if (member type '(:int32 :uint32 :int64 :uint64 :sint32 :sint64
                      :fixed32 :sfixed32 :fixed64 :sfixed64
                      :string :bytes :bool :float :double))
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0))
+    `(locally (declare #.$optimize-serialization
                        (type (simple-array (unsigned-byte 8)) ,buffer)
                        (type fixnum ,index))
        ,(ecase type
-          ((:int32 :uint32)
+          ((:int32 :int64)
+           `(decode-int ,buffer ,index))
+          ((:uint32)
            `(decode-uint32 ,buffer ,index))
-          ((:int64 :uint64)
+          ((:uint64)
            `(decode-uint64 ,buffer ,index))
           ((:sint32)
            `(multiple-value-bind (val idx)
    Watch out, this function turns off most type checking and all array bounds checking."
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (locally (declare #.$optimize-serialization)
     (multiple-value-bind (len idx)
         (decode-uint32 buffer index)
       (declare (type (unsigned-byte 32) len)
               (return-from deserialize-packed (values values idx)))
             (multiple-value-bind (val nidx)
                 (ecase type
-                  ((:int32 :uint32)
+                  ((:int32 :int64)
+                   (decode-int buffer idx))
+                  ((:uint32)
                    (decode-uint32 buffer idx))
-                  ((:int64 :uint64)
+                  ((:uint64)
                    (decode-uint64 buffer idx))
                   ((:sint32)
-                   (multiple-value-bind (val idx)
+                   (multiple-value-bind (val nidx)
                        (decode-uint32 buffer idx)
-                     (values (zig-zag-decode32 val) idx)))
+                     (values (zig-zag-decode32 val) nidx)))
                   ((:sint64)
-                   (multiple-value-bind (val idx)
+                   (multiple-value-bind (val nidx)
                        (decode-uint64 buffer idx)
-                     (values (zig-zag-decode64 val) idx)))
+                     (values (zig-zag-decode64 val) nidx)))
                   ((:fixed32)
                    (decode-fixed32 buffer idx))
                   ((:sfixed32)
                    (decode-fixed64 buffer idx))
                   ((:sfixed64)
                    (decode-sfixed64 buffer idx))
+                  ((:bool)
+                   (multiple-value-bind (val nidx)
+                       (decode-uint32 buffer idx)
+                     (values (if (i= val 0) nil t) nidx)))
                   ((:float)
                    (decode-single buffer idx))
                   ((:double)
               (setq idx nidx))))))))
 
 (define-compiler-macro deserialize-packed (&whole form type buffer index)
+  (setq type (fold-symbol type))
   (if (member type '(:int32 :uint32 :int64 :uint64 :sint32 :sint64
                      :fixed32 :sfixed32 :fixed64 :sfixed64
-                     :float :double))
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0))
+                     :bool :float :double))
+    `(locally (declare #.$optimize-serialization
                        (type (simple-array (unsigned-byte 8)) ,buffer)
                        (type fixnum ,index))
        (block deserialize-packed
                    (return-from deserialize-packed (values values idx)))
                  (multiple-value-bind (val nidx)
                      ,(ecase type
-                        ((:int32 :uint32)
+                        ((:int32 :int64)
+                         `(decode-int ,buffer idx))
+                        ((:uint32)
                          `(decode-uint32 ,buffer idx))
-                        ((:int64 :uint64)
+                        ((:uint64)
                          `(decode-uint64 ,buffer idx))
                         ((:sint32)
-                         `(multiple-value-bind (val idx)
+                         `(multiple-value-bind (val nidx)
                               (decode-uint32 ,buffer idx)
-                            (values (zig-zag-decode32 val) idx)))
+                            (values (zig-zag-decode32 val) nidx)))
                         ((:sint64)
-                         `(multiple-value-bind (val idx)
+                         `(multiple-value-bind (val nidx)
                               (decode-uint64 ,buffer idx)
-                            (values (zig-zag-decode64 val) idx)))
+                            (values (zig-zag-decode64 val) nidx)))
                         ((:fixed32)
                          `(decode-fixed32 ,buffer idx))
                         ((:sfixed32)
                          `(decode-fixed64 ,buffer idx))
                         ((:sfixed64)
                          `(decode-sfixed64 ,buffer idx))
+                        ((:bool)
+                         `(multiple-value-bind (val nidx)
+                              (decode-uint32 ,buffer idx)
+                            (values (if (i= val 0) nil t) nidx)))
                         ((:float)
                          `(decode-single ,buffer idx))
                         ((:double)
                    (setq idx nidx))))))))
     form))
 
-(defun deserialize-enum (values buffer index)
-  "Deserializes the next enum value take from 'values'.
+(defun deserialize-enum (enum-values buffer index)
+  "Deserializes the next enum value take from 'enum-values'.
    Deserializes from the byte vector 'buffer' starting at 'index'.
    Returns the value and and the new index into the buffer.
    Watch out, this function turns off most type checking and all array bounds checking."
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
+  (declare (type list enum-values)
+           (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (locally (declare #.$optimize-serialization)
     (multiple-value-bind (val idx)
-        (decode-uint32 buffer index)
-      (let ((val (let ((e (find val values :key #'proto-index)))
+        (decode-int buffer index)
+      (let ((val (let ((e (find val enum-values :key #'proto-index)))
                    (and e (proto-value e)))))
         (values val idx)))))
 
+(defun deserialize-packed-enum (enum-values buffer index)
+  "Deserializes the next packed enum values given in 'enum-values'.
+   Deserializes from the byte vector 'buffer' starting at 'index'.
+   Returns the value and and the new index into the buffer.
+   Watch out, this function turns off most type checking and all array bounds checking."
+  (declare (type list enum-values)
+           (type (simple-array (unsigned-byte 8)) buffer)
+           (type fixnum index))
+  (locally (declare #.$optimize-serialization)
+    (multiple-value-bind (len idx)
+        (decode-uint32 buffer index)
+      (declare (type (unsigned-byte 32) len)
+               (type fixnum idx))
+      (let ((end (i+ idx len)))
+        (declare (type (unsigned-byte 32) end))
+        (with-collectors ((values collect-value))
+          (loop
+            (when (>= idx end)
+              (return-from deserialize-packed-enum (values values idx)))
+            (multiple-value-bind (val nidx)
+                (decode-int buffer idx)
+              (let ((val (let ((e (find val enum-values :key #'proto-index)))
+                           (and e (proto-value e)))))
+                (collect-value val)
+                (setq idx nidx)))))))))
+
 
 ;;; Object sizing
 
   "Returns the size in bytes that the primitive object will take when serialized.
    Watch out, this function turns off most type checking."
   (declare (type (unsigned-byte 32) tag))
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (locally (declare #.$optimize-serialization)
     (ecase type
-      ((:int32 :uint32)
-       (i+ (length32 tag) (length32 val)))
-      ((:int64 :uint64)
-       (i+ (length32 tag) (length64 val)))
+      ((:int32 :uint32 :int64 :uint64)
+       (i+ (varint-length tag) (varint-length val)))
       ((:sint32)
-       (i+ (length32 tag) (length32 (zig-zag-encode32 val))))
+       (i+ (varint-length tag) (varint-length (zig-zag-encode32 val))))
       ((:sint64)
-       (i+ (length32 tag) (length64 (zig-zag-encode64 val))))
+       (i+ (varint-length tag) (varint-length (zig-zag-encode64 val))))
       ((:fixed32 :sfixed32)
-       (i+ (length32 tag) 4))
+       (i+ (varint-length tag) 4))
       ((:fixed64 :sfixed64)
-       (i+ (length32 tag) 8))
+       (i+ (varint-length tag) 8))
       ((:string)
        (let ((len (babel:string-size-in-octets val :encoding :utf-8)))
-         (i+ (length32 tag) (length32 len) len)))
+         (i+ (varint-length tag) (varint-length len) len)))
       ((:bytes)
        (let ((len (length val)))
-         (i+ (length32 tag) (length32 len) len)))
+         (i+ (varint-length tag) (varint-length len) len)))
       ((:bool)
-       (i+ (length32 tag) 1))
+       (i+ (varint-length tag) 1))
       ((:float)
-       (i+ (length32 tag) 4))
+       (i+ (varint-length tag) 4))
       ((:double)
-       (i+ (length32 tag) 8))
+       (i+ (varint-length tag) 8))
       ;; A few of our homegrown types
       ((:symbol)
        (let ((len (if (keywordp val)
                     (length (symbol-name val))
                     (i+ (length (package-name (symbol-package val))) 1 (length (symbol-name val))))))
-         (i+ (length32 tag) (length32 len) len)))
+         (i+ (varint-length tag) (varint-length len) len)))
       ((:date :time :datetime :timestamp)
-       (i+ (length32 tag) 8)))))
+       (i+ (varint-length tag) 8)))))
 
 (define-compiler-macro prim-size (&whole form val type tag)
+  (setq type (fold-symbol type)
+        tag  (fold-symbol tag))
   (if (member type '(:int32 :uint32 :int64 :uint64 :sint32 :sint64
                      :fixed32 :sfixed32 :fixed64 :sfixed64
                      :string :bytes :bool :float :double))
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+    `(locally (declare #.$optimize-serialization)
        ,(ecase type
-          ((:int32 :uint32)
-           `(i+ (length32 ,tag) (length32 ,val)))
-          ((:int64 :uint64)
-           `(i+ (length32 ,tag) (length64 ,val)))
+          ((:int32 :int64 :uint32 :uint64)
+           `(i+ (varint-length ,tag) (varint-length ,val)))
           ((:sint32)
-           `(i+ (length32 ,tag) (length32 (zig-zag-encode32 ,val))))
+           `(i+ (varint-length ,tag) (varint-length (zig-zag-encode32 ,val))))
           ((:sint64)
-           `(i+ (length32 ,tag) (length64 (zig-zag-encode64 ,val))))
+           `(i+ (varint-length ,tag) (varint-length (zig-zag-encode64 ,val))))
           ((:fixed32 :sfixed32)
-           `(i+ (length32 ,tag) 4))
+           `(i+ (varint-length ,tag) 4))
           ((:fixed64 :sfixed64)
-           `(i+ (length32 ,tag) 8))
+           `(i+ (varint-length ,tag) 8))
           ((:string)
            `(let ((len (babel:string-size-in-octets ,val :encoding :utf-8)))
-              (i+ (length32 ,tag) (length32 len) len)))
+              (i+ (varint-length ,tag) (varint-length len) len)))
           ((:bytes)
            `(let ((len (length ,val)))
-              (i+ (length32 ,tag) (length32 len) len)))
+              (i+ (varint-length ,tag) (varint-length len) len)))
           ((:bool)
-           `(i+ (length32 ,tag) 1))
+           `(i+ (varint-length ,tag) 1))
           ((:float)
-           `(i+ (length32 ,tag) 4))
+           `(i+ (varint-length ,tag) 4))
           ((:double)
-           `(i+ (length32 ,tag) 8))))
+           `(i+ (varint-length ,tag) 8))))
     form))
 
-(defun packed-size (values type tag)
+(defun packed-size (values type tag &optional vectorp)
   "Returns the size in bytes that the packed object will take when serialized.
    Watch out, this function turns off most type checking."
-  (declare (type (unsigned-byte 32) tag))
-  (locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare (ignore vectorp)
+           (type (unsigned-byte 32) tag))
+  (locally (declare #.$optimize-serialization)
     (let ((len (let ((len 0))
                  (declare (type fixnum len))
-                 (dolist (val values len)
-                   (iincf len (ecase type
-                                ((:int32 :uint32) (length32 val))
-                                ((:int64 :uint64) (length64 val))
-                                ((:sint32) (length32 (zig-zag-encode32 val)))
-                                ((:sint64) (length64 (zig-zag-encode64 val)))
-                                ((:fixed32 :sfixed32) 4)
-                                ((:fixed64 :sfixed64) 8)
-                                ((:float) 4)
-                                ((:double) 8)))))))
+                 (map () #'(lambda (val)
+                             (iincf len (ecase type
+                                          ((:int32 :uint32 :int64 :uint64) (varint-length val))
+                                          ((:sint32) (varint-length (zig-zag-encode32 val)))
+                                          ((:sint64) (varint-length (zig-zag-encode64 val)))
+                                          ((:fixed32 :sfixed32) 4)
+                                          ((:fixed64 :sfixed64) 8)
+                                          ((:bool)   1)
+                                          ((:float)  4)
+                                          ((:double) 8)))) values)
+                 len)))
       (declare (type (unsigned-byte 32) len))
       ;; Two value: the full size of the packed object, and the size
       ;; of just the payload
-      (values (i+ (length32 tag) (length32 len) len) len))))
-
-(define-compiler-macro packed-size (&whole form values type tag)
-  (if (member type '(:int32 :uint32 :int64 :uint64 :sint32 :sint64
-                     :fixed32 :sfixed32 :fixed64 :sfixed64
-                     :float :double))
-    `(locally (declare (optimize (speed 3) (safety 0) (debug 0)))
+      (values (i+ (varint-length tag) (varint-length len) len) len))))
+
+;; The optimized serializers supply 'vectorp' so we can generate better code
+(define-compiler-macro packed-size (&whole form values type tag
+                                    &optional (vectorp nil vectorp-p))
+  (setq type (fold-symbol type)
+        tag  (fold-symbol tag))
+  (if (and vectorp-p
+           (member type '(:int32 :uint32 :int64 :uint64 :sint32 :sint64
+                          :fixed32 :sfixed32 :fixed64 :sfixed64
+                          :bool :float :double)))
+    `(locally (declare #.$optimize-serialization)
        (let ((len (let ((len 0))
                     (declare (type fixnum len))
-                    (dolist (val ,values len)
-                      (iincf len ,(ecase type
-                                    ((:int32 :uint32) `(length32 val))
-                                    ((:int64 :uint64) `(length64 val))
-                                    ((:sint32) `(length32 (zig-zag-encode32 val)))
-                                    ((:sint64) `(length64 (zig-zag-encode64 val)))
-                                    ((:fixed32 :sfixed32) `4)
-                                    ((:fixed64 :sfixed64) `8)
-                                    ((:float) `4)
-                                    ((:double) `8)))))))
+                    (,(if vectorp 'dovector 'dolist) (val ,values)
+                       (iincf len ,(ecase type
+                                     ((:int32 :uint32 :int64 :uint64) `(varint-length val))
+                                     ((:sint32) `(varint-length (zig-zag-encode32 val)))
+                                     ((:sint64) `(varint-length (zig-zag-encode64 val)))
+                                     ((:fixed32 :sfixed32) `4)
+                                     ((:fixed64 :sfixed64) `8)
+                                     ((:bool)   `1)
+                                     ((:float)  `4)
+                                     ((:double) `8))))
+                    len)))
          (declare (type (unsigned-byte 32) len))
-         (values (i+ (length32 (the (unsigned-byte 32) ,tag)) (length32 len) len) len)))
+         (values (i+ (varint-length (the (unsigned-byte 32) ,tag)) (varint-length len) len) len)))
     form))
 
-(defun enum-size (val values tag)
+(defun enum-size (val enum-values tag)
   "Returns the size in bytes that the enum object will take when serialized."
-  (declare (type (unsigned-byte 32) tag))
-  (let ((idx (let ((e (find val values :key #'proto-value)))
+  (declare (type list enum-values)
+           (type (unsigned-byte 32) tag))
+  (let ((idx (let ((e (find val enum-values :key #'proto-value)))
                (and e (proto-index e)))))
-    (assert idx () "There is no enum value for ~S" val)
-    (i+ (length32 tag) (length32 idx))))
+    (unless idx
+      (serialization-error "There is no enum value for ~S" val))
+    (i+ (varint-length tag) (varint-length (ldb (byte 32 0) idx)))))
+
+(defun packed-enum-size (values enum-values tag)
+  "Returns the size in bytes that the enum values will take when serialized."
+  (declare (type list enum-values)
+           (type (unsigned-byte 32) tag))
+  (let ((len (let ((len 0))
+               (declare (type fixnum len))
+               (map () #'(lambda (val)
+                           (let ((idx (let ((e (find val enum-values :key #'proto-value)))
+                                        (and e (proto-index e)))))
+                             (unless idx
+                               (serialization-error "There is no enum value for ~S" val))
+                             (iincf len (varint-length (ldb (byte 32 0) idx))))) values)
+               len)))
+    (declare (type (unsigned-byte 32) len))
+    ;; Two value: the full size of the packed object, and the size
+    ;; of just the payload
+    (values (i+ (varint-length tag) (varint-length len) len) len)))
 
 \f
 ;;; Wire-level encoders
 ;;; These are called at the lowest level, so arg types are assumed to be correct
 
-(defun encode-uint32 (val buffer index)
-  "Encodes the unsigned 32-bit integer 'val' as a varint into the buffer
-   at the given index.
+(defmacro generate-integer-encoders (bits)
+  "Generate 32- or 64-bit versions of integer encoders."
+  (assert (and (plusp bits) (zerop (mod bits 8))))
+  (let* ((encode-uint   (fintern "~A~A" 'encode-uint bits))
+         (encode-fixed  (fintern "~A~A" 'encode-fixed bits))
+         (encode-sfixed (fintern "~A~A" 'encode-sfixed bits))
+         (bytes (/ bits 8))
+         ;; Given bits, can we use fixnums safely?
+         (fixnump (<= bits (integer-length most-negative-fixnum)))
+         (ldb (if fixnump 'ildb 'ldb))
+         (ash (if fixnump 'iash 'ash))
+         (zerop-val (if fixnump '(i= val 0) '(zerop val))))
+    `(progn
+       (defun ,encode-uint (val buffer index)
+         ,(format nil
+                  "Encodes the unsigned ~A-bit integer 'val' as a varint into the buffer at the given index.~
+                   ~&    Modifies the buffer, and returns the new index into the buffer.~
+                   ~&    Watch out, this function turns off all type checking and array bounds checking." bits)
+         (declare #.$optimize-serialization)
+         (let ((val (ldb (byte ,bits 0) val)))
+           (declare (type (unsigned-byte ,bits) val)
+                    (type (simple-array (unsigned-byte 8)) buffer)
+                    (type fixnum index))
+           ;; Seven bits at a time, least significant bits first
+           (loop do (let ((bits (,ldb (byte 7 0) val)))
+                      (declare (type (unsigned-byte 8) bits))
+                      (setq val (,ash val -7))
+                      (setf (aref buffer index)
+                            (ilogior bits (if ,zerop-val 0 128)))
+                      (iincf index))
+                 until ,zerop-val))
+         (values index buffer))                 ;return the buffer to improve 'trace'
+       (defun ,encode-fixed (val buffer index)
+         ,(format nil
+                  "Encodes the unsigned ~A-bit integer 'val' as a fixed int into the buffer at the given index.~
+                   ~&    Modifies the buffer, and returns the new index into the buffer.~
+                   ~&    Watch out, this function turns off all type checking and array bounds checking." bits)
+         (declare #.$optimize-serialization)
+         (declare (type (unsigned-byte ,bits) val)
+                  (type (simple-array (unsigned-byte 8)) buffer)
+                  (type fixnum index))
+         (loop repeat ,bytes doing
+           (let ((byte (,ldb (byte 8 0) val)))
+             (declare (type (unsigned-byte 8) byte))
+             (setq val (,ash val -8))
+             (setf (aref buffer index) byte)
+             (iincf index)))
+         (values index buffer))
+       (defun ,encode-sfixed (val buffer index)
+         ,(format nil
+                  "Encodes the signed ~A-bit integer 'val' as a fixed int into the buffer at the given index.~
+                   ~&    Modifies the buffer, and returns the new index into the buffer.~
+                   ~&    Watch out, this function turns off all type checking and array bounds checking." bits)
+         (declare #.$optimize-serialization)
+         (declare (type (signed-byte ,bits) val)
+                  (type (simple-array (unsigned-byte 8)) buffer)
+                  (type fixnum index))
+         (loop repeat ,bytes doing
+           (let ((byte (,ldb (byte 8 0) val)))
+             (declare (type (unsigned-byte 8) byte))
+             (setq val (,ash val -8))
+             (setf (aref buffer index) byte)
+             (iincf index)))
+         (values index buffer)))))
+
+(generate-integer-encoders 32)
+(generate-integer-encoders 64)
+
+(defun encode-int (val buffer index)
+  "Encodes the signed integer 'val' as a varint into the buffer at the given index.
    Modifies the buffer, and returns the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (unsigned-byte 32) val)
-           (type (simple-array (unsigned-byte 8)) buffer)
+  (declare #.$optimize-serialization)
+  (declare (type (simple-array (unsigned-byte 8)) buffer)
+           (type (signed-byte 64) val)
            (type fixnum index))
   ;; Seven bits at a time, least significant bits first
-  (loop do (let ((bits (ildb (byte 7 0) val)))
-             (declare (type (unsigned-byte 8) bits))
-             (setq val (iash val -7))
-             (setf (aref buffer index) (ilogior bits (if (i= val 0) 0 128)))
-             (iincf index))
-        until (i= val 0))
-  (values index buffer))                        ;return the buffer to improve 'trace'
-
-(defun encode-uint64 (val buffer index)
-  "Encodes the unsigned 64-bit integer 'val' as a varint into the buffer
-   at the given index.
-   Modifies the buffer, and returns the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (unsigned-byte 64) val)
-           (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  (loop do (let ((bits (ldb (byte 7 0) val)))
-             (declare (type (unsigned-byte 8) bits))
-             (setq val (ash val -7))
-             (setf (aref buffer index) (ilogior bits (if (zerop val) 0 128)))
-             (iincf index))
-        until (zerop val))
-  (values index buffer))
-
-(defun encode-fixed32 (val buffer index)
-  "Encodes the unsigned 32-bit integer 'val' as a fixed int into the buffer
-   at the given index.
-   Modifies the buffer, and returns the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (unsigned-byte 32) val)
-           (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  (loop repeat 4 doing
-    (let ((byte (ildb (byte 8 0) val)))
-      (declare (type (unsigned-byte 8) byte))
-      (setq val (iash val -8))
-      (setf (aref buffer index) byte)
-      (iincf index)))
-  (values index buffer))
-
-(defun encode-fixed64 (val buffer index)
-  "Encodes the unsigned 64-bit integer 'val' as a fixed int into the buffer
-   at the given index.
-   Modifies the buffer, and returns the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (unsigned-byte 64) val)
-           (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  (loop repeat 8 doing
-    (let ((byte (ldb (byte 8 0) val)))
-      (declare (type (unsigned-byte 8) byte))
-      (setq val (ash val -8))
-      (setf (aref buffer index) byte)
-      (iincf index)))
-  (values index buffer))
-
-(defun encode-sfixed32 (val buffer index)
-  "Encodes the signed 32-bit integer 'val' as a fixed int into the buffer
-   at the given index.
-   Modifies the buffer, and returns the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (signed-byte 32) val)
-           (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  (loop repeat 4 doing
-    (let ((byte (ildb (byte 8 0) val)))
-      (declare (type (unsigned-byte 8) byte))
-      (setq val (iash val -8))
-      (setf (aref buffer index) byte)
-      (iincf index)))
-  (values index buffer))
-
-(defun encode-sfixed64 (val buffer index)
-  "Encodes the signed 64-bit integer 'val' as a fixed int into the buffer
-   at the given index.
-   Modifies the buffer, and returns the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (signed-byte 64) val)
-           (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  (loop repeat 8 doing
-    (let ((byte (ldb (byte 8 0) val)))
-      (declare (type (unsigned-byte 8) byte))
-      (setq val (ash val -8))
-      (setf (aref buffer index) byte)
-      (iincf index)))
-  (values index buffer))
+  (loop repeat 9                ;up to 63 bits
+        do (setf (aref buffer index) (ldb (byte 7 0) val))
+           (setq val (ash val -7))
+        until (zerop val)
+        do (iincf (aref buffer index) #x80)
+           (iincf index)
+        finally (unless (zerop val)     ;take the 64th bit as needed
+                  (setf (aref buffer index) 1)
+                  (unless (= val -1)
+                    (serialization-error "Integer too large while encoding VarInt."))))
+  (values (iincf index) buffer))        ;return the buffer to improve 'trace'
 
 (defun encode-single (val buffer index)
   "Encodes the single float 'val' into the buffer at the given index.
    Modifies the buffer, and returns the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type single-float val)
            (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
   (let ((bits (single-float-bits val)))
+    (declare (type (signed-byte 32) bits))
     (loop repeat 4 doing
       (let ((byte (ldb (byte 8 0) bits)))
         (declare (type (unsigned-byte 8) byte))
   "Encodes the double float 'val' into the buffer at the given index.
    Modifies the buffer, and returns the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type double-float val)
            (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
   (multiple-value-bind (low high)
       (double-float-bits val)
+    (declare (type (unsigned-byte 32) low)
+             (type (signed-byte 32) high))
     (loop repeat 4 doing
       (let ((byte (ldb (byte 8 0) low)))
         (declare (type (unsigned-byte 8) byte))
   "Encodes the octets into the buffer at the given index.
    Modifies the buffer, and returns the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
   (let* ((octets (babel:string-to-octets string :encoding :utf-8))
          (len (length octets))
          (idx (encode-uint32 len buffer index)))
-    (declare (type fixnum len)
+    (declare (type (simple-array (unsigned-byte 8)) octets)
+             (type fixnum len)
              (type (unsigned-byte 32) idx))
     (replace buffer octets :start1 idx)
     (values (i+ idx len) buffer)))
   "Encodes the octets into the buffer at the given index.
    Modifies the buffer, and returns the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
+  (declare #.$optimize-serialization)
+  (declare (type (array (unsigned-byte 8)) octets)
+           (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
   (let* ((len (length octets))
          (idx (encode-uint32 len buffer index)))
 
 ;; Decode the value from the buffer at the given index,
 ;; then return the value and new index into the buffer
-(defun decode-uint32 (buffer index)
-  "Decodes the next 32-bit varint integer in the buffer at the given index.
+(defmacro generate-integer-decoders (bits)
+  "Generate 32- or 64-bit versions of integer decoders."
+  (assert (and (plusp bits) (zerop (mod bits 8))))
+  (let* ((decode-uint (fintern "~A~A" 'decode-uint bits))
+         (decode-fixed  (fintern "~A~A" 'decode-fixed bits))
+         (decode-sfixed (fintern "~A~A" 'decode-sfixed bits))
+         (bytes (/ bits 8))
+         ;; Given bits, can we use fixnums safely?
+         (fixnump (<= bits (integer-length most-negative-fixnum)))
+         (ash (if fixnump 'iash 'ash))
+         (decf (if fixnump 'idecf 'decf))
+         (logior (if fixnump 'ilogior 'logior))
+         (logbitp (if fixnump 'ilogbitp 'logbitp)))
+    `(progn
+       (defun ,decode-uint (buffer index)
+         ,(format nil
+                  "Decodes the next ~A-bit varint integer in the buffer at the given index.~
+                   ~&    Returns both the decoded value and the new index into the buffer.~
+                   ~&    Watch out, this function turns off all type checking and array bounds checking." bits)
+         (declare #.$optimize-serialization)
+         (declare (type (simple-array (unsigned-byte 8)) buffer)
+                  (type fixnum index))
+         ;; Seven bits at a time, least significant bits first
+         (let ((val 0))
+           (declare (type (unsigned-byte ,bits) val))
+           (loop for places fixnum upfrom 0 by 7
+                 for byte fixnum = (prog1 (aref buffer index) (iincf index))
+                 do (let ((bits (ildb (byte 7 0) byte)))
+                      (declare (type (unsigned-byte 8) bits))
+                      (setq val (,logior val (,ash bits places))))
+                 while (ilogbitp 7 byte)
+                 finally (progn
+                           (unless (< val ,(ash 1 bits))
+                             (serialization-error "The value ~D is longer than ~A bits" val ,bits))
+                           (return (values val index))))))
+       (defun ,decode-fixed (buffer index)
+         ,(format nil
+                  "Decodes the next ~A-bit unsigned fixed integer in the buffer at the given index.~
+                   ~&    Returns both the decoded value and the new index into the buffer.~
+                   ~&    Watch out, this function turns off all type checking and array bounds checking." bits)
+         (declare #.$optimize-serialization)
+         (declare (type (simple-array (unsigned-byte 8)) buffer)
+                  (type fixnum index))
+         ;; Eight bits at a time, least significant bits first
+         (let ((val 0))
+           ,@(when fixnump `((declare (type fixnum val))))
+           (loop repeat ,bytes
+                 for places fixnum upfrom 0 by 8
+                 for byte fixnum = (prog1 (aref buffer index) (iincf index))
+                 do (setq val (,logior val (,ash byte places))))
+           (values val index)))
+       (defun ,decode-sfixed (buffer index)
+         ,(format nil
+                  "Decodes the next ~A-bit signed fixed integer in the buffer at the given index.~
+                   ~&    Returns both the decoded value and the new index into the buffer.~
+                   ~&    Watch out, this function turns off all type checking and array bounds checking." bits)
+         (declare #.$optimize-serialization)
+         (declare (type (simple-array (unsigned-byte 8)) buffer)
+                  (type fixnum index))
+         ;; Eight bits at a time, least significant bits first
+         (let ((val 0))
+           ,@(when fixnump `((declare (type fixnum val))))
+           (loop repeat ,bytes
+                 for places fixnum upfrom 0 by 8
+                 for byte fixnum = (prog1 (aref buffer index) (iincf index))
+                 do (setq val (,logior val (,ash byte places))))
+           (when (,logbitp ,(1- bits) val)      ;sign bit set, so negative value
+             (,decf val ,(ash 1 bits)))
+           (values val index))))))
+
+(generate-integer-decoders 32)
+(generate-integer-decoders 64)
+
+(defun decode-int (buffer index)
+  "Decodes the next varint integer in the buffer at the given index.
    Returns both the decoded value and the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
-  ;; Seven bits at a time, least significant bits first
-  (loop with val fixnum = 0
-        for places fixnum upfrom 0 by 7
-        for byte fixnum = (prog1 (aref buffer index) (iincf index))
-        do (setq val (ilogior val (iash (ildb (byte 7 0) byte) places)))
-        until (i< byte 128)
-        finally (progn
-                  (assert (< val #.(ash 1 32)) ()
-                          "The value ~D is longer than 32 bits" val)
-                  (return (values val index)))))
-
-(defun decode-uint64 (buffer index)
-  "Decodes the next 64-bit varint integer in the buffer at the given index.
-   Returns both the decoded value and the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  ;; Seven bits at a time, least significant bits first
-  (loop with val = 0
-        for places fixnum upfrom 0 by 7
-        for byte fixnum = (prog1 (aref buffer index) (iincf index))
-        do (setq val (logior val (ash (ildb (byte 7 0) byte) places)))
-        until (i< byte 128)
-        finally (return (values val index))))
-
-(defun decode-fixed32 (buffer index)
-  "Decodes the next 32-bit unsigned fixed integer in the buffer at the given index.
-   Returns both the decoded value and the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  ;; Eight bits at a time, least significant bits first
-  (let ((val 0))
-    (declare (type fixnum val))
-    (loop repeat 4
-          for places fixnum upfrom 0 by 8
-          for byte fixnum = (prog1 (aref buffer index) (iincf index))
-          do (setq val (ilogior val (iash byte places))))
-    (values val index)))
-
-(defun decode-sfixed32 (buffer index)
-  "Decodes the next 32-bit signed fixed integer in the buffer at the given index.
-   Returns both the decoded value and the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  ;; Eight bits at a time, least significant bits first
-  (let ((val 0))
-    (declare (type fixnum val))
-    (loop repeat 4
-          for places fixnum upfrom 0 by 8
-          for byte fixnum = (prog1 (aref buffer index) (iincf index))
-          do (setq val (ilogior val (iash byte places))))
-    (when (i= (ldb (byte 1 31) val) 1)              ;sign bit set, so negative value
-      (decf val #.(ash 1 32)))
-    (values val index)))
-
-(defun decode-fixed64 (buffer index)
-  "Decodes the next unsigned 64-bit fixed integer in the buffer at the given index.
-   Returns both the decoded value and the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  ;; Eight bits at a time, least significant bits first
-  (let ((val 0))
-    (loop repeat 8
-          for places fixnum upfrom 0 by 8
-          for byte fixnum = (prog1 (aref buffer index) (iincf index))
-          do (setq val (logior val (ash byte places))))
-    (values val index)))
-
-(defun decode-sfixed64 (buffer index)
-  "Decodes the next signed 64-bit fixed integer in the buffer at the given index.
-   Returns both the decoded value and the new index into the buffer.
-   Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (simple-array (unsigned-byte 8)) buffer)
-           (type fixnum index))
-  ;; Eight bits at a time, least significant bits first
-  (let ((val 0))
-    (loop repeat 8
-          for places fixnum upfrom 0 by 8
-          for byte fixnum = (prog1 (aref buffer index) (iincf index))
-          do (setq val (logior val (ash byte places))))
-    (when (i= (ldb (byte 1 63) val) 1)             ;sign bit set, so negative value
-      (decf val #.(ash 1 64)))
-    (values val index)))
+  (multiple-value-bind (val index)
+      (decode-uint64 buffer index)
+    (declare (type (unsigned-byte 64) val))
+    (values (if (logbitp 63 val)
+                (- val #.(ash 1 64))
+                val)
+            index)))
 
 (defun decode-single (buffer index)
   "Decodes the next single float in the buffer at the given index.
    Returns both the decoded value and the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
   ;; Eight bits at a time, least significant bits first
   (let ((bits 0))
+    (declare (type (unsigned-byte 32) bits))
     (loop repeat 4
           for places fixnum upfrom 0 by 8
           for byte fixnum = (prog1 (aref buffer index) (iincf index))
           do (setq bits (logior bits (ash byte places))))
-    (when (i= (ldb (byte 1 31) bits) 1)             ;sign bit set, so negative value
-      (decf bits #.(ash 1 32)))
-    (values (make-single-float bits) index)))
+    (values (make-single-float (if (logbitp 31 bits)    ;sign bit
+                                 (- bits #.(ash 1 32))
+                                 bits))
+            index)))
 
 (defun decode-double (buffer index)
   "Decodes the next double float in the buffer at the given index.
    Returns both the decoded value and the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
   ;; Eight bits at a time, least significant bits first
   (let ((low  0)
         (high 0))
+    (declare (type (unsigned-byte 32) low)
+             (type (unsigned-byte 32) high))
     (loop repeat 4
           for places fixnum upfrom 0 by 8
           for byte fixnum = (prog1 (aref buffer index) (iincf index))
           for byte fixnum = (prog1 (aref buffer index) (iincf index))
           do (setq high (logior high (ash byte places))))
     ;; High bits are signed, but low bits are unsigned
-    (when (i= (ldb (byte 1 31) high) 1)             ;sign bit set, so negative value
-      (decf high #.(ash 1 32)))
-    (values (make-double-float low high) index)))
+    (values (make-double-float low (if (logbitp 31 high)        ;sign bit
+                                     (- high #.(ash 1 32))
+                                     high))
+            index)))
 
 (defun decode-string (buffer index)
   "Decodes the next UTF-8 encoded string in the buffer at the given index.
    Returns both the decoded string and the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
   (multiple-value-bind (len idx)
   "Decodes the next octets in the buffer at the given index.
    Returns both the decoded value and the new index into the buffer.
    Watch out, this function turns off all type checking and array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index))
   (multiple-value-bind (len idx)
 ;;; Wire-level lengths
 ;;; These are called at the lowest level, so arg types are assumed to be correct
 
-(defun length32 (val)
-  "Returns the length that 'val' will take when encoded as a 32-bit integer."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (unsigned-byte 32) val))
-  (let ((size 0))
-    (declare (type fixnum size))
-    (loop do (progn
-               (setq val (iash val -7))
-               (iincf size))
-          until (i= val 0))
-    size))
-
-(defun length64 (val)
-  "Returns the length that 'val' will take when encoded as a 64-bit integer."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
-  (declare (type (unsigned-byte 64) val))
-  (let ((size 0))
-    (declare (type fixnum size))
-    (loop do (progn
-               (setq val (ash val -7))
-               (iincf size))
-          until (zerop val))
-    size))
-
+(defun varint-length (val)
+  "Return the length that 'val' will take when encoded as a varint integer."
+  (declare #.$optimize-serialization)
+  (loop repeat 10                       ;max length of varint
+        do (setq val (ash val -7))
+        count 1
+        until (zerop val)))
 
 ;;; Skipping elements
 ;;; This is called at the lowest level, so arg types are assumed to be correct
   "Skip an element in the buffer at the index of the given wire type.
    Returns the new index in the buffer.
    Watch out, this function turns off all type checking and all array bounds checking."
-  (declare (optimize (speed 3) (safety 0) (debug 0)))
+  (declare #.$optimize-serialization)
   (declare (type (simple-array (unsigned-byte 8)) buffer)
            (type fixnum index)
            (type (unsigned-byte 32) tag))
                    ((i= (i- tag $wire-type-start-group) (i- new-tag $wire-type-end-group))
                     (return idx))
                    (t
-                    (assert (i= (i- tag $wire-type-start-group) (i- new-tag $wire-type-end-group)) ()
-                            "Couldn't find a matching end group tag"))))))
+                    (unless (i= (i- tag $wire-type-start-group) (i- new-tag $wire-type-end-group))
+                      (serialization-error "Couldn't find a matching end group tag")))))))
     (t index)))