From: Vladimír Vondruš <mosra@centrum.cz>
Date: Sun, 4 Feb 2018 22:26:55 +0000 (+0100)
Subject: doxygen: simplify search (de)serialization code.
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~cjwatson/git?a=commitdiff_plain;h=03eb5b5a15c8ddbffe70827ea0cb1e7dc29e17fe;p=blog.git

doxygen: simplify search (de)serialization code.
---

diff --git a/doxygen/dox2html5.py b/doxygen/dox2html5.py
index 32fee0c9..ab9a7b32 100755
--- a/doxygen/dox2html5.py
+++ b/doxygen/dox2html5.py
@@ -178,8 +178,7 @@ class ResultMap:
     #
     offset_struct = struct.Struct('<I')
     flags_struct = struct.Struct('<B')
-    prefix_struct = struct.Struct('<H')
-    prefix_length_struct = struct.Struct('<B')
+    prefix_struct = struct.Struct('<HB')
     suffix_length_struct = struct.Struct('<B')
 
     def __init__(self):
@@ -302,8 +301,7 @@ class ResultMap:
         # Write the entries themselves
         for e in self.entries:
             if e.flags & ResultFlag.HAS_PREFIX:
-                output += self.prefix_struct.pack(e.prefix)
-                output += self.prefix_length_struct.pack(e.prefix_length)
+                output += self.prefix_struct.pack(e.prefix, e.prefix_length)
             if e.flags & ResultFlag.HAS_SUFFIX:
                 output += self.suffix_length_struct.pack(e.suffix_length)
             output += e.name.encode('utf-8')
diff --git a/doxygen/test/test_search.py b/doxygen/test/test_search.py
index f2837596..7a43c346 100755
--- a/doxygen/test/test_search.py
+++ b/doxygen/test/test_search.py
@@ -133,11 +133,11 @@ def pretty_print_map(serialized: bytes, colors=False):
         flags = ResultFlag(ResultMap.flags_struct.unpack_from(serialized, i*4 + 3)[0])
         extra = []
         if flags & ResultFlag.HAS_PREFIX:
-            extra += ['prefix={}[:{}]'.format(ResultMap.prefix_struct.unpack_from(serialized, offset)[0] & 0x00ffffff, ResultMap.prefix_length_struct.unpack_from(serialized, offset + 2)[0])]
-            offset += 3
+            extra += ['prefix={}[:{}]'.format(*ResultMap.prefix_struct.unpack_from(serialized, offset))]
+            offset += ResultMap.prefix_struct.size
         if flags & ResultFlag.HAS_SUFFIX:
             extra += ['suffix_length={}'.format(ResultMap.suffix_length_struct.unpack_from(serialized, offset)[0])]
-            offset += 1
+            offset += ResultMap.suffix_length_struct.size
         if flags & ResultFlag.DEPRECATED:
             extra += ['deprecated']
         if flags & ResultFlag.DELETED: